Changes

[InstCombine] Fold uadd.sat(a, b) == 0 and usub.sat(a, b) == 0
This adds folds for comparing uadd.sat/usub.sat with zero:
 * uadd.sat(a, b) == 0 => a == 0 && b == 0 => (a | b) == 0
* usub.sat(a, b) == 0 => a <= b
And inverted forms for !=.
Differential Revision: https://reviews.llvm.org/D69224
llvm-svn: 375374

Reverted r375254 as it has broken some build bots for a long time.
llvm-svn: 375375

gn build: Merge r375375
llvm-svn: 375376

[NFC][InstCombine] conditional sign-extend of high-bit-extract: 'and'
pat. can be 'or' pattern.
In this pattern, all the "magic" bits that we'd add are all high sign
bits, and in the value we'd be adding to they are all unset, not
unexpectedly, so we can have an `or` there: 
https://rise4fun.com/Alive/ups
llvm-svn: 375377

[InstCombine] conditional sign-extend of high-bit-extract: 'or' pattern.
In this pattern, all the "magic" bits that we'd `add` are all high sign
bits, and in the value we'd be adding to they are all unset, not
unexpectedly, so we can have an `or` there: 
https://rise4fun.com/Alive/ups
It is possible that `haveNoCommonBitsSet()` should be taught about this 
pattern so that we never have an `add` variant, but the reasoning would 
need to be recursive (because of that `select`), so i'm not really sure 
that would be worth it just yet.
llvm-svn: 375378

[IndVars] Eliminate loop exits with equivalent exit counts
We can end up with two loop exits whose exit counts are equivalent, but
whose textual representation is different and non-obvious. For the
sub-case where we have a series of exits which dominate one another
(common), eliminate any exits which would iterate *after* a previous
exit on the exiting iteration.
As noted in the TODO being removed, I'd always thought this was a good
idea, but I've now seen this in a real workload as well.
Interestingly, in review, Nikita pointed out there's let another
oppurtunity to leverage SCEV's reasoning.  If we kept track of the min
of dominanting exits so far, we could discharge exits with EC >= MDE. 
This is less powerful than the existing transform (since later exits
aren't considered), but potentially more powerful for any case where
SCEV can prove a >= b, but neither a == b or a > b.  I don't have an
example to illustrate that oppurtunity, but won't be suprised if we find
one and return to handle that case as well.  
Differential Revision: https://reviews.llvm.org/D69009
llvm-svn: 375379

[IndVars] Add a todo to reflect a further oppurtunity identified in
D69009
Nikita pointed out an oppurtunity, might as well document it in the
code.
llvm-svn: 375380

[X86] Check Subtarget.hasSSE3() before calling shouldUseHorizontalOp and
emitting X86ISD::FHADD in LowerUINT_TO_FP_i64.
This was a regression from r375341.
Fixes PR43729.
llvm-svn: 375381

[Attributor] Teach AANoCapture to use information in-flight more
aggressively
AAReturnedValues, AAMemoryBehavior, and AANoUnwind, can provide 
information that helps during the tracking or even justifies no-capture. 
We now use this information and enable no-capture in some test cases 
designed a long while a ago for these cases.
llvm-svn: 375382

[Attributor][FIX] Silence sign-compare warning
llvm-svn: 375384

[BPF] fix indirect call assembly code
Currently, for indirect call, the assembly code printed out as
 callx <imm> This is not right, it should be
 callx <reg>
Fixed the issue with proper format.
Differential Revision: https://reviews.llvm.org/D69229
llvm-svn: 375386

[IR] Fix mayReadFromMemory() for writeonly calls
Current implementation of Instruction::mayReadFromMemory() returns
!doesNotAccessMemory() which is !ReadNone. This does not take into
account that the writeonly attribute also indicates that the call does
not read from memory.
The patch changes the predicate to !doesNotReadMemory() that reflects
the intended behavior.
Differential Revision: https://reviews.llvm.org/D69086
llvm-svn: 375389

[LLD] Move duplicated dwarf parsing code to the Common library. NFC.
Differential Revision: https://reviews.llvm.org/D69197
llvm-svn: 375390

[LLD] [COFF] Use the local dwarf code instead of Symbolizer for
resolving code locations. NFC.
As we now have code that parses the dwarf info for variable locations, 
we can use that instead of relying on the higher level Symbolizer
library, reducing the previous two different dwarf codepaths into one.
Differential Revision: https://reviews.llvm.org/D69198
llvm-svn: 375391

[LLDB] [Windows] Initial support for ARM register contexts
Differential Revision: https://reviews.llvm.org/D69226
llvm-svn: 375392

gn build: Merge r375390
llvm-svn: 375393

[InstCombine] Allow values with multiple users in
SimplifyDemandedVectorElts
Summary: Allow for ignoring the check for a single use in
SimplifyDemandedVectorElts to be able to simplify operands if
DemandedElts is known to contain the union of elements used by all
users. It is a responsibility of a caller of SimplifyDemandedVectorElts
to supply correct DemandedElts.
Simplify a series of extractelement instructions if only a subset of 
elements is used.
Reviewers: reames, arsenm, majnemer, nhaehnle
Reviewed By: nhaehnle
Subscribers: wdng, jvesely, nhaehnle, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D67345
llvm-svn: 375395

[CVP] Deduce no-wrap on `mul`
Summary:
`ConstantRange::makeGuaranteedNoWrapRegion()` knows how to deal with
`mul` since rL335646, there is exhaustive test coverage. This is already
used by CVP's `processOverflowIntrinsic()`, and by SCEV's
`StrengthenNoWrapFlags()`
That being said, currently, this doesn't help much in the end:
| statistic                              |     old |     new | delta |
percentage |
| correlated-value-propagation.NumMulNSW |       4 |     275 |   271 |  
6775.00% |
| correlated-value-propagation.NumMulNUW |       4 |    1323 |  1319 | 
32975.00% |
| correlated-value-propagation.NumMulNW  |       8 |    1598 |  1590 | 
19875.00% |
| correlated-value-propagation.NumNSW    |    5715 |    5986 |   271 |  
   4.74% |
| correlated-value-propagation.NumNUW    |    9193 |   10512 |  1319 |  
  14.35% |
| correlated-value-propagation.NumNW     |   14908 |   16498 |  1590 |  
  10.67% |
| instcount.NumAddInst                   |  275871 |  275869 |    -2 |  
   0.00% |
| instcount.NumBrInst                    |  708234 |  708232 |    -2 |  
   0.00% |
| instcount.NumMulInst                   |   43812 |   43810 |    -2 |  
   0.00% |
| instcount.NumPHIInst                   |  316786 |  316784 |    -2 |  
   0.00% |
| instcount.NumTruncInst                 |   62165 |   62167 |     2 |  
   0.00% |
| instcount.NumUDivInst                  |    2528 |    2526 |    -2 |  
  -0.08% |
| instcount.TotalBlocks                  |  842995 |  842993 |    -2 |  
   0.00% |
| instcount.TotalInsts                   | 7376486 | 7376478 |    -8 |  
   0.00% |
(^ test-suite plain, tests still pass)
Reviewers: nikic, reames, luqmana, sanjoy, timshen
Reviewed By: reames
Subscribers: hiraditya, javed.absar, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69203
llvm-svn: 375396

[NFC][InstCombine] Fixup comments
As noted in post-commit review of rL375378375378.
llvm-svn: 375397

Use Align for TFL::TransientStackAlignment
Summary: This is patch is part of a series to introduce an Alignment
type. See this thread for context:
http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html See this
patch for the introduction of the type: https://reviews.llvm.org/D64790
Reviewers: courbet
Subscribers: arsenm, dschuff, jyknight, sdardis, jvesely, nhaehnle,
sbc100, jgravelle-google, hiraditya, aheejin, fedor.sergeev, jrtc27,
atanasyan, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69216
llvm-svn: 375398

[LLD] [COFF] Fix use of uninitialized memory since SVN r375390
llvm-svn: 375400

[ARM] Add and adjust saturation tests for upcoming qadd changes. NFC
llvm-svn: 375401

[ARM] Lower sadd_sat to qadd8 and qadd16
Lower the target independent signed saturating intrinsics to qadd8 and
qadd16. This custom lowers them from a sadd_sat, catching the node early
before it is promoted. It also adds a QADD8b and QADD16b node to mean
the bottom "lane" of a qadd8/qadd16, so that we can call demand bits on
it to show that it does not use the upper bits.
Also handles QSUB8 and QSUB16.
Differential Revision: https://reviews.llvm.org/D68974
llvm-svn: 375402

[MemCpyOpt] Fixing Incorrect Code Motion while Handling Aggregate Type
Values
Summary: When MemCpyOpt is handling aggregate type values, if an
instruction (let's call it P) between the targeting load (L) and store
(S) clobbers the source pointer of L, it will try to hoist S before P.
This process will also hoist S's data dependency instructions.
However, the current implementation has a bug that if one of S's
dependency instructions is //also// a user of P, MemCpyOpt will not
prevent it from being hoisted above P and cause a use-before-define
error. For example, in the newly added test file (i.e.
`aggregate-type-crash.ll`), it will try to hoist both `store %my_struct
%1, %my_struct* %3` and its dependent, `%3 = bitcast i8* %2 to
%my_struct*`, above `%2 = call i8* @my_malloc(%my_struct* %0)`. Creating
the following BB:
``` entry:
 %1 = bitcast i8* %4 to %my_struct*
 %2 = bitcast %my_struct* %1 to i8*
 %3 = bitcast %my_struct* %0 to i8*
 call void @llvm.memcpy.p0i8.p0i8.i64(i8* align 4 %2, i8* align 4 %3,
i64 8, i1 false)
 %4 = call i8* @my_malloc(%my_struct* %0)
 ret void
``` Where there is a use-before-define error between `%1` and `%4`.
Update: The compiler for the Pony Programming Language [also encounter
the same bug](https://github.com/ponylang/ponyc/issues/3140)
Patch by Min-Yih Hsu (myhsu)
Reviewers: eugenis, pcc, dblaikie, dneilson, t.p.northover, lattner
Reviewed By: eugenis
Subscribers: lenary, hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D66060
llvm-svn: 375403

[obj2yaml] - Stop triggering UB when dumping corrupted strings.
We have a following code to find quote type:
if (isspace(S.front()) || isspace(S.back()))
...
Problem is that:
"int isspace( int ch ): The behavior is undefined if the value of ch is
not representable as unsigned char and is not equal to EOF."
(https://en.cppreference.com/w/cpp/string/byte/isspace)
This patch shows how this UB can be triggered and fixes an issue.
Differential revision: https://reviews.llvm.org/D69160
llvm-svn: 375404

[obj2yaml] - Fix a comment. NFC.
I forgot to address this nit before committing..
llvm-svn: 375405

Fix llvm signal tests build.
llvm-svn: 375406

[Alignment][NFC] TargetCallingConv::setOrigAlign and
TargetLowering::getABIAlignmentForCallingConv
Summary: This is patch is part of a series to introduce an Alignment
type. See this thread for context:
http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html See this
patch for the introduction of the type: https://reviews.llvm.org/D64790
Reviewers: courbet
Subscribers: sdardis, hiraditya, jrtc27, atanasyan, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69243
llvm-svn: 375407

[llvm/Object] - Make ELFObjectFile::getRelocatedSection return
Expected<section_iterator>
It returns just a section_iterator currently and have a
report_fatal_error call inside. This change adds a way to return errors
and handle them on caller sides.
The patch also changes/improves current users and adds test cases.
Differential revision: https://reviews.llvm.org/D69167
llvm-svn: 375408

Simplify usage of setFPAttrs.
In some cases using the return value of setFPAttrs simplifies the code. 
In other cases it complicates the code with ugly casts, so stop doing 
it. NFC.
llvm-svn: 375409

[Alignment][NFC] TargetCallingConv::setByValAlign
Summary: This is patch is part of a series to introduce an Alignment
type. See this thread for context:
http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html See this
patch for the introduction of the type: https://reviews.llvm.org/D64790
Reviewers: courbet
Subscribers: hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69248
llvm-svn: 375410

[ARM] Add qadd lowering from a sadd_sat
This lowers a sadd_sat to a qadd by treating it as legal. Also adds qsub
at the same time.
The qadd instruction sets the q flag, but we already have many cases
where we do not model this in llvm.
Differential Revision: https://reviews.llvm.org/D68976
llvm-svn: 375411

[docs][llvm-ar] Update llvm-ar command guide
The llvm-ar command guide had not been updated in some time, it was 
missing current functionality and contained information that was out of
date. This change:
- Updates the use of reStructuredText directives, as seen in other tools
 command guides.
- Updates the command synopsis.
- Updates the descriptions of the tool behaviour.
- Updates the options section.
- Adds details of MRI script functionality.
- Removes the sections "Standards" and "File Format"
Differential Revision: https://reviews.llvm.org/D68998
llvm-svn: 375412

[Alignment][NFC] Add a helper function to DataLayout
Summary: This is patch is part of a series to introduce an Alignment
type. See this thread for context:
http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html See this
patch for the introduction of the type: https://reviews.llvm.org/D64790
Reviewers: courbet
Subscribers: llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69258
llvm-svn: 375413

[ARM] Extra qdadd patterns
This adds some new qdadd patterns to go along with the other recently
added qadd's.
Differential Revision: https://reviews.llvm.org/D68999
llvm-svn: 375414

[lldb] drop .symtab removal in minidebuginfo tests
Summary: After D69041, we no longer have to manually remove the .symtab
section once yaml2obj was run.
Reviewers: espindola, alexshap
Subscribers: emaste, arichardson, MaskRay, lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D69254
llvm-svn: 375415

[Alignment][NFC] Instructions::getLoadStoreAlignment
Summary: This is patch is part of a series to introduce an Alignment
type. See this thread for context:
http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html See this
patch for the introduction of the type: https://reviews.llvm.org/D64790
Subscribers: hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69256
llvm-svn: 375416

[Types] Define a getWithNewBitWidth for Types and make use of it
This is designed to change the bitwidth of a type without altering the
number of vector lanes. Also useful in D68651. Otherwise an NFC.
Differential Revision: https://reviews.llvm.org/D69139
llvm-svn: 375417

Pre-commit test cases for D64713.
llvm-svn: 375418

[Alignment][NFC] Finish transition for `Loads`
Summary: This is patch is part of a series to introduce an Alignment
type. See this thread for context:
http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html See this
patch for the introduction of the type: https://reviews.llvm.org/D64790
Reviewers: courbet
Subscribers: hiraditya, asbirlea, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69253
llvm-svn: 375419

[NFC] Cleanup with variable name IsPPC64 & IsDarwin
Clean up PPCAsmPrinter with IsPPC64 and IsDarwin.
Differential Revision: https://reviews.llvm.org/D69259
llvm-svn: 375420

Fix Polly
llvm-svn: 375421

[lldb] Add test for executing static initializers in expression command
llvm-svn: 375422

PCH debug info: Avoid appending the source directory to an absolute path
When building a precompiled header in -fmodule-format=obj (i.e.,
`-gmodules) in an absolute path, the locig in 
CGDebugInfo::createCompileUnit would unconditionally append the source 
directory to the -main-file-name. This patch avoids that behavior for 
absolute paths.
rdar://problem/46045865
Differential Revision: https://reviews.llvm.org/D69213
llvm-svn: 375423

[AArch64][DebugInfo] Do not recompute CalleeSavedStackSize (Take 2)
Commit message from D66935:
This patch fixes a bug exposed by D65653 where a subsequent invocation 
of `determineCalleeSaves` ends up with a different size for the callee 
save area, leading to different frame-offsets in debug information.
In the invocation by PEI, `determineCalleeSaves` tries to determine 
whether it needs to spill an extra callee-saved register to get an 
emergency spill slot. To do this, it calls 'estimateStackSize' and 
manually adds the size of the callee-saves to this. PEI then allocates 
the spill objects for the callee saves and the remaining frame layout is
calculated accordingly.
A second invocation in LiveDebugValues causes estimateStackSize to
return the size of the stack frame including the callee-saves. Given
that the size of the callee-saves is added to this, these callee-saves
are counted twice, which leads `determineCalleeSaves` to believe the
stack has become big enough to require spilling an extra callee-save as
emergency spillslot. It then updates CalleeSavedStackSize with a larger
value.
Since CalleeSavedStackSize is used in the calculation of the frame 
offset in getFrameIndexReference, this leads to incorrect offsets for 
variables/locals when this information is recalculated after PEI.
This patch fixes the lldb unit tests in
`functionalities/thread/concurrent_events/*`
Changes after D66935:
Ensures AArch64FunctionInfo::getCalleeSavedStackSize does not return the
uninitialized CalleeSavedStackSize when running `llc` on a specific pass
where the MIR code has already been expected to have gone through PEI.
Instead, getCalleeSavedStackSize (when passed the MachineFrameInfo) will
try to recalculate the CalleeSavedStackSize from the CalleeSavedInfo. In
debug mode, the compiler will assert the recalculated size equals the
cached size as calculated through a call to determineCalleeSaves.
This fixes two tests:
 test/DebugInfo/AArch64/asan-stack-vars.mir
 test/DebugInfo/AArch64/compiler-gen-bbs-livedebugvalues.mir that
otherwise fail when compiled using msan.
Reviewed By: omjavaid, efriedma
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D68783
llvm-svn: 375425

IndVarSimplify - silence static analyzer dyn_cast<> null dereference
warning. NFCI.
The static analyzer is warning about a potential null dereference, but
we should be able to use cast<> directly and if not assert will fire for
us.
llvm-svn: 375426

CrossDSOCFI - silence static analyzer dyn_cast<> null dereference
warning. NFCI.
The static analyzer is warning about a potential null dereference, but
we should be able to use cast<> directly and if not assert will fire for
us.
llvm-svn: 375427

GuardWidening - silence static analyzer null dereference warning with
assertion. NFCI.
llvm-svn: 375428

GVNHoist - silence static analyzer dyn_cast<> null dereference warning
in hasEHOrLoadsOnPath call. NFCI.
The static analyzer is warning about a potential null dereference, but
we should be able to use cast<> directly and if not assert will fire for
us.
llvm-svn: 375429

SystemZISelLowering - supportedAddressingMode - silence static analyzer
dyn_cast<> null dereference warning. NFCI.
The static analyzer is warning about a potential null dereference, but
we should be able to use cast<> directly and if not assert will fire for
us.
llvm-svn: 375430

Unify timeouts in gdbserver tests and ensure they are larger if ASAN is
enabled.
llvm-svn: 375431

[NFC] Add missing include to fix modules build
This header doesn't seem to be parsable on its own and breaks the module
build therefore with the following error:
While building module 'LLVM_Backend' imported from
llvm-project/llvm/lib/CodeGen/MachineScheduler.cpp:14: In file included
from <module-includes>:62: 
llvm-project/llvm/include/llvm/CodeGen/MachinePipeliner.h:91:20: error:
declaration of 'AAResultsWrapperPass' must be imported from module
'LLVM_Analysis.AliasAnalysis' before it is required
   AU.addRequired<AAResultsWrapperPass>();
                  ^ 
llvm-project/llvm/include/llvm/Analysis/AliasAnalysis.h:1157:7: note:
previous declaration is here class AAResultsWrapperPass : public
FunctionPass {
     ^ llvm-project/llvm/lib/CodeGen/MachineScheduler.cpp:14:10: fatal
error: could not build module 'LLVM_Backend'
~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 2 errors generated.
llvm-svn: 375433

Fix Wdocumentation warning. NFCI.
llvm-svn: 375434

[PowerPC] Regenerate test for D52431
llvm-svn: 375435

Prune Pass.h include from DataLayout.h. NFCI
Summary: Reduce include dependencies by no longer including Pass.h from 
DataLayout.h. That include seemed irrelevant to DataLayout, as well as
being irrelevant to several users of DataLayout.
Reviewers: rnk
Reviewed By: rnk
Subscribers: mehdi_amini, hiraditya, cfe-commits, llvm-commits
Tags: #clang, #llvm
Differential Revision: https://reviews.llvm.org/D69261
llvm-svn: 375436

Prune include of DataLayout.h from include/clang/Basic/TargetInfo.h. NFC
Summary: Use a forward declaration of DataLayout instead of including 
DataLayout.h in clangs TargetInfo.h. This reduces include dependencies
toward DataLayout.h (and other headers such as DerivedTypes.h, Type.h
that is included by DataLayout.h).
Needed to move implemantation of TargetInfo::resetDataLayout from
TargetInfo.h to TargetInfo.cpp.
Reviewers: rnk
Reviewed By: rnk
Subscribers: jvesely, nhaehnle, cfe-commits, llvm-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D69262
llvm-svn: 375438

[test] Merge Driver/as-w-warnings.c into as-no-warnings.c
For -integrated-as RUN lines we can remove -target.
llvm-svn: 375439

SemaExceptionSpec - silence static analyzer getAs<> null dereference
warnings. NFCI.
The static analyzer is warning about potential null dereferences, but in
these cases we should be able to use castAs<> directly and if not assert
will fire for us.
llvm-svn: 375440

[NFC][CVP] Add `shl` no-wrap deduction test coverage
llvm-svn: 375441

[Clang] Add VerboseOutputStream to CompilerInstance
Remove one instance of a hardcoded output stream in 
CompilerInstance::ExecuteAction. There are still other cases of output 
being hard-coded to standard streams in ExecuteCompilerInvocation, but 
this patch covers the case when no flags like -version or -help are 
passed, namely the "X warnings and Y errors generated." diagnostic.
Differential Revision: https://reviews.llvm.org/D53768
llvm-svn: 375442

SemaTemplateDeduction - silence static analyzer getAs<> null dereference
warnings. NFCI.
The static analyzer is warning about potential null dereferences, but in
these cases we should be able to use castAs<> directly and if not assert
will fire for us.
llvm-svn: 375443

Reverted r375425 as it broke some buildbots.
llvm-svn: 375444

[X86] Rename matchBitOpReduction to matchScalarReduction. NFCI.
This doesn't need to be just for bitops, but the ops do need to be fully
associative.
llvm-svn: 375445

[AMDGPU] Select AGPR in PHI operand legalization
If a PHI defines AGPR legalize its operands to AGPR. At the moment we
can get an AGPR PHI with VGPR operands. I am not aware of any problems
as it seems to be handled gracefully in RA, but this is not right
anyway.
It also slightly decreases VGPR pressure in some cases because we do not
have to a copy via VGPR.
Differential Revision: https://reviews.llvm.org/D69206
llvm-svn: 375446

AMDGPU: Slightly restructure m0 init code
This will allow using another operation to produce the glue in a future
change.
llvm-svn: 375447

AMDGPU: Stop adding m0 implicit def to SGPR spills
r375293 removed the SGPR spilling with scalar stores path, so this is no
longer necessary. This also always had the defect of adding the def even
when this path wasn't in use.
llvm-svn: 375448

AMDGPU: Erase redundant redefs of m0 in SIFoldOperands
Only handle simple inter-block redefs of m0 to the same value. This 
avoids interference from redefs of m0 in SILoadStoreOptimzer. I was 
initially teaching that pass to ignore redefs of m0, but having them not
exist beforehand is much simpler.
This is in preparation for deleting the current special m0 handling in 
SIFixSGPRCopies to allow the register coalescer to handle the difficult
cases.
llvm-svn: 375449

AMDGPU: Use CopyToReg for interp intrinsic lowering
This doesn't use the default value, so doesn't benefit from the hack to
help optimize it.
llvm-svn: 375450

[GISel][CombinerHelper] Add a combine turning shuffle_vector into
concat_vectors
Teach the CombinerHelper how to turn shuffle_vectors, that concatenate
vectors, into concat_vectors and add this combine to the AArch64
pre-legalizer combiner.
Differential Revision: https://reviews.llvm.org/D69149
llvm-svn: 375452

[clang-fuzzer] Add new fuzzer target for Objective-C
Summary:
- Similar to that of `clang-fuzzer` itself but instead only
 targets Objective-C source files via cc1
- Also adds an example corpus directory containing some
 input for Objective-C
Subscribers: mgorny, jfb, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D69171
llvm-svn: 375453

Found more timeouts to unify.
llvm-svn: 375454

[CVP] No-wrap deduction for `shl`
Summary: This is the last `OverflowingBinaryOperator` for which we don't
deduce flags. D69217 taught
`ConstantRange::makeGuaranteedNoWrapRegion()` about it.
The effect is better than of the `mul` patch (D69203):
| statistic                              |     old |     new | delta | %
change |
| correlated-value-propagation.NumAddNUW |    7145 |    7144 |    -1 |
-0.0140% |
| correlated-value-propagation.NumAddNW  |   12126 |   12125 |    -1 |
-0.0082% |
| correlated-value-propagation.NumAnd    |     443 |     446 |     3 | 
0.6772% |
| correlated-value-propagation.NumNSW    |    5986 |    7158 |  1172 |
19.5790% |
| correlated-value-propagation.NumNUW    |   10512 |   13304 |  2792 |
26.5601% |
| correlated-value-propagation.NumNW     |   16498 |   20462 |  3964 |
24.0272% |
| correlated-value-propagation.NumShlNSW |       0 |    1172 |  1172 |  
       |
| correlated-value-propagation.NumShlNUW |       0 |    2793 |  2793 |  
       |
| correlated-value-propagation.NumShlNW  |       0 |    3965 |  3965 |  
       |
| instcount.NumAShrInst                  |   13824 |   13790 |   -34 |
-0.2459% |
| instcount.NumAddInst                   |  277584 |  277586 |     2 | 
0.0007% |
| instcount.NumAndInst                   |   66061 |   66056 |    -5 |
-0.0076% |
| instcount.NumBrInst                    |  709153 |  709147 |    -6 |
-0.0008% |
| instcount.NumICmpInst                  |  483709 |  483708 |    -1 |
-0.0002% |
| instcount.NumSExtInst                  |   79497 |   79496 |    -1 |
-0.0013% |
| instcount.NumShlInst                   |   40691 |   40654 |   -37 |
-0.0909% |
| instcount.NumSubInst                   |   61997 |   61996 |    -1 |
-0.0016% |
| instcount.NumZExtInst                  |   68208 |   68211 |     3 | 
0.0044% |
| instcount.TotalBlocks                  |  843916 |  843910 |    -6 |
-0.0007% |
| instcount.TotalInsts                   | 7387528 | 7387448 |   -80 |
-0.0011% |
Reviewers: nikic, reames, sanjoy, timshen
Reviewed By: nikic
Subscribers: hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69277
llvm-svn: 375455

[lit] Remove redundancy from names and comments
llvm-svn: 375456

AMDGPU: Select basic interp directly from intrinsics
llvm-svn: 375457

[lit] Simplify test scheduling via multiprocessing.Pool
llvm-svn: 375458

XFAIL TestLocalVariables.py on Windows
This test has been failing for a while on the Windows bot.
https://bugs.llvm.org/show_bug.cgi?id=43752
llvm-svn: 375459

AMDGPU/GlobalISel: Legalize fast unsafe FDIV
Reviewers: arsenm
Reviewed By: arsenm
Subscribers: kzhuravl, jvesely, wdng, nhaehnle, yaxunl, rovka,
dstuttard, tpr, t-tye, hiraditya, Petar.Avramovic, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69231
llvm-svn: 375460

[examples] Add a dependency on ExecutionEngine to LLJITWithObjectCache
example.
ExecutionEngine.cpp contains the anchor() for the ObjectCache base
class, so we need an explicit dependency on it.
Patch by Stephen Neuendorffer. Thanks Stephen!
llvm-svn: 375461

[X86][SSE] Add OR(EXTRACTELT(X,0),OR(EXTRACTELT(X,1))) movmsk v2X64
tests
llvm-svn: 375462

[X86][SSE] Add OR(EXTRACTELT(X,0),OR(EXTRACTELT(X,1))) -> MOVMSK+CMP
reduction combine
llvm-svn: 375463

Factor out common test functionality into a helper class. (NFC)
llvm-svn: 375464

whitespace cleanup
llvm-svn: 375465

[Implicit Modules] Add -cc1 option -fmodules-strict-context-hash which
includes search paths and diagnostics.
This is a recommit of r375322 and r375327 with a fix for the Windows
test breakage.
llvm-svn: 375466

Fix -fuse-init-array decision logic on NetBSD
For NetBSD 9 and later, it is the default. On older versions, only ARM 
and AArch64 use it by default.
llvm-svn: 375468

[NFC] Fix typos in CMake comment
llvm-svn: 375469

[X86][BMI] Pull out schedule classes from bmi_andn<> and bmi_bls<>
Stop hardwiring classes
llvm-svn: 375470

[hwasan] Workaround unwinder issues in try-catch test.
Android links the unwinder library to every DSO. The problem is, 
unwinder has global state, and hwasan implementation of personality 
function wrapper happens to rub it the wrong way.
Switch the test to static libc++ as a temporary workaround.
llvm-svn: 375471

Fix lld detection in standalone compiler-rt.
Summary: Right now all hwasan tests on Android are silently disabled
because they require "has_lld" and standalone compiler-rt can not (and
AFAIK was never able to) set it.
Reviewers: pcc, dyung
Subscribers: dberris, mgorny, #sanitizers, llvm-commits
Tags: #sanitizers, #llvm
Differential Revision: https://reviews.llvm.org/D69196
llvm-svn: 375472

[lit] Move increase_process_limit to ParallelRun
Increasing the process limit only makes sense when we use multiple 
processes.
llvm-svn: 375474

remove multi-argument form of PythonObject::Reset()
Summary: With this patch, only the no-argument form of `Reset()` remains
in PythonDataObjects.   It also deletes PythonExceptionState in favor of 
PythonException, because the only call-site of PythonExceptionState was 
also using Reset, so I cleaned up both while I was there.
Reviewers: JDevlieghere, clayborg, labath, jingham
Reviewed By: labath
Subscribers: mgorny, lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D69214
llvm-svn: 375475

fix PythonDataObjectsTest.TestExceptions on windows
Looks like on windows googlemock regexes treat newlines differently from
on darwin.    This patch fixes the regex in this test so it will work on
both.
Fixes: https://reviews.llvm.org/D69214 llvm-svn: 375477

Minor coding style fix. NFC.
llvm-svn: 375478

Refactor DependencyScanningTool to its own file
Summary: There's no behavior change - just moving DependencyScanningTool
to its own file since this tool can be reused across both
clang-scan-deps binary and an interface exposed as part of libClang
APIs.
Reviewers: arphaman, jkorous, Bigcheese, dexonsmith
Subscribers: mgorny, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D69186
llvm-svn: 375483

gn build: Merge r375483
llvm-svn: 375484

[CMake] Allow overriding MSVC_DIA_SDK_DIR via CMake
This eases using it in cross-msvc setups.
Differential Revision: https://reviews.llvm.org/D69239
llvm-svn: 375485

[CMake] [WinMsvc] Look for includes and libs in ${MSVC_BASE}/atlmfc
This is necessary if building with the DIA SDK enabled.
Differential Revision: https://reviews.llvm.org/D69240
llvm-svn: 375486

[clang-fuzzer] Update proto fuzzer example for r375453.
llvm-svn: 375487

[FrontendTests] Don't actually run the full compiler, parsing is
sufficient.
llvm-svn: 375488

Revert "[FrontendTests] Don't actually run the full compiler, parsing is
sufficient."
This reverts commit 375488.
llvm-svn: 375489

[builtins][test] Avoid unportable mmap call in clear_cache_test.c
Within the last two weeks, the Builtins-*-sunos :: clear_cache_test.c
started to FAIL on Solaris.  Running it under truss shows
  mmap(0x00000000, 128, PROT_READ|PROT_WRITE|PROT_EXEC,
MAP_PRIVATE|MAP_ANON, 0, 0) Err#22 EINVAL
 _exit(1)
While there are several possible reasons mmap can return EINVAL on
Solaris, it turns out it's this one (from mmap(2)):
  MAP_ANON  was  specified,  but the file descriptor was not
  -1.
And indeed even the Linux mmap(2) documents this as unportable:
  MAP_ANONYMOUS
         The mapping is not backed by any file; its contents are
initial‐
         ized to zero.  The fd argument is ignored; however, some 
imple‐
         mentations require fd to be -1 if MAP_ANONYMOUS (or MAP_ANON)
is
         specified, and portable applications should  ensure  this.  
The
This patch follows this advise.  Tested on x86_64-pc-linux-gnu,
amd64-pc-solaris2.11 and sparcv9-sun-solaris2.11.
Differential Revision: https://reviews.llvm.org/D68455
llvm-svn: 375490

[FrontendTests] Try again to make test not write an output file
Setting the output stream to nulls seems to work.
llvm-svn: 375491

[LLVMDebugInfoPDB] - Use cantFail() instead of assert().
Currently injected-sources-native.test fails with "Expected<T> value was
in success state.
(Note: Expected<T> values in success mode must still be checked prior to
being destroyed)" when llvm is compiled with
LLVM_ENABLE_ABI_BREAKING_CHECKS in Release.
The problem is that getStringForID returns Expected<StringRef> and
Expected value must always be checked, even if it is in success state. 
Checking with assert only helps in Debug and is wrong.
Differential revision: https://reviews.llvm.org/D69251
llvm-svn: 375492

[ThinLTO] Don't internalize during promotion
Differential revision: https://reviews.llvm.org/D69107
llvm-svn: 375493

[Alignment][NFC] Attributes use Align/MaybeAlign
Summary: This is patch is part of a series to introduce an Alignment
type. See this thread for context:
http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html See this
patch for the introduction of the type: https://reviews.llvm.org/D64790
Reviewers: courbet
Subscribers: jholewinski, hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69278
llvm-svn: 375495

[Alignment][NFC] Use MaybeAlign in AttrBuilder
Summary: This is patch is part of a series to introduce an Alignment
type. See this thread for context:
http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html See this
patch for the introduction of the type: https://reviews.llvm.org/D64790
Reviewers: courbet
Subscribers: hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69300
llvm-svn: 375496

[PowerPC] Turn on CR-Logical reducer pass
This re-commits r375152 which was pulled in r375233 because it broke the
EXPENSIVE_CHECKS bot on Windows.
The reason for the failure was a bug in the pass that the commit turned 
on by default. This patch fixes that bug and turns the pass back on. 
This patch has been verified on the buildbot that originally failed 
thanks to Simon Pilgrim.
Differential revision: https://reviews.llvm.org/D52431
llvm-svn: 375497

[Alignment][NFC] Convert LoadInst to MaybeAlign
Summary: This is patch is part of a series to introduce an Alignment
type. See this thread for context:
http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html See this
patch for the introduction of the type: https://reviews.llvm.org/D64790
Reviewers: courbet
Subscribers: hiraditya, jfb, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69302
llvm-svn: 375498

[Alignment][NFC] Convert StoreInst to MaybeAlign
Summary: This is patch is part of a series to introduce an Alignment
type. See this thread for context:
http://lists.llvm.org/pipermail/llvm-dev/2019-July/133851.html See this
patch for the introduction of the type: https://reviews.llvm.org/D64790
Reviewers: courbet
Subscribers: hiraditya, jfb, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69303
llvm-svn: 375499

[ThinLTO] Add code comment. NFC
llvm-svn: 375500

[MIPS GlobalISel] Select MSA vector generic and builtin add
Select vector G_ADD for MIPS32 with MSA. We have to set bank for vector
operands to fprb and selectImpl will do the rest.
__builtin_msa_addv_<format> will be transformed into G_ADD in
legalizeIntrinsic and selected in the same way.
__builtin_msa_addvi_<format> will be directly selected into 
ADDVI_<format> in legalizeIntrinsic. MIR tests for it have unnecessary
additional copies. Capture current state of tests with
run-pass=legalizer with a test in test/CodeGen/MIR/Mips.
Differential Revision: https://reviews.llvm.org/D68984
llvm-svn: 375501

[MIParser] Set RegClassOrRegBank during instruction parsing
MachineRegisterInfo::createGenericVirtualRegister sets RegClassOrRegBank
to static_cast<RegisterBank *>(nullptr). MIParser on the other hand
doesn't. When we attempt to constrain Register Class on such VReg,
additional COPY is generated. This way we avoid COPY instructions
showing in test that have MIR input while they are not present with
llvm-ir input that was used to create given MIR for a -run-pass test.
Differential Revision: https://reviews.llvm.org/D68946
llvm-svn: 375502

[InstCombine] Signed saturation tests. NFC
llvm-svn: 375503

    [libcxx] Remove shared_ptr::make_shared
    Summary: This patch removes `shared_ptr::make_shared` as it is not
part of the standard. This patch also adds __create_with_cntrl_block,
which is a help function that can be used in std::allocate_shared and
std::make_shared. This is the third patch (out of 4) from D66178.
    Reviewers: EricWF, mclow.lists, ldionne
    Subscribers: christof, dexonsmith, libcxx-commits
    Tags: #libc
    Differential Revision: https://reviews.llvm.org/D68805
llvm-svn: 375504

[InstCombine] Signed saturation patterns
This adds an instcombine matcher for code that attempts to perform
signed saturating arithmetic by casting to a higher type. Unsigned cases
are already matched, this adds extra matches for the more complex signed
cases, which involves matching the min(max(add a b)) nodes with proper
extends to ensure legality.
Differential Revision: https://reviews.llvm.org/D68651
llvm-svn: 375505

Update git-llvm script to push to GitHub
Summary: Note: This patch should not be pushed until SVN has become
read-only. It should be the first patch committed directly to GitHub.
This patch updates git-llvm to check for merge commits and then push 
changes to GitHub if none are found.  All logic related to SVN has been 
removed.
Reviewers: jyknight
Subscribers: lenary, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D67772

[lldb] Adjust for the new class_rw_t layout.
The field holding the "ro" will now be a union. If the low bit is set, 
then it isn't an ro and it needs to be dereferenced once more to get to 
it. If the low bit isn't set, then it is a proper class_ro_t
No dedicated test is needed as this code path will trigger when running 
the existing Objective-C tests under a current version of the runtime.

gn build: make sync build work with git revs now that svn is gone

[AMDGPU] Allow tied operand subreg folding
Turns out it makes sense, contrarily to what comment said.
Differential Revision: https://reviews.llvm.org/D69287

Test commit - add clarification to README regarding Darwin.

Relax assertions when there's really no entries. [NFC]

[AMDGPU] Updated fold-vgpr-copy.mir test. NFC.

[NFC][LVI][CVP] Tests where pre-specified `add` no-wrap flags could be
used by LVI
There's `ConstantRange::addWithNoWrap()`, LVI could use it to further 
constrain the range, if an `add` already has some no-wrap flags
specified.

[DAGCombine][ARM] x ==/!= c  ->  (x - c) ==/!= 0  iff '-c' can be folded
into the x node.
Summary: This fold, helps recover from the rest of the D62266 ARM
regressions. https://rise4fun.com/Alive/TvpC
Note that while the fold is quite flexible, i've restricted it to the
single interesting pattern at the moment.
Reviewers: efriedma, craig.topper, spatel, RKSimon, deadalnix
Reviewed By: deadalnix
Subscribers: javed.absar, kristof.beyls, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D62450

[HIP] Add option -fgpu-allow-device-init
Add this option to allow device side class type global variables with
non-trivial ctor/dtor. device side init/fini functions will be emitted,
which will be executed by HIP runtime when the fat binary is
loaded/unloaded.
This feature is to facilitate implementation of device side sanitizer
which requires global vars with non-trival ctors.
By default this option is disabled.
Differential Revision: https://reviews.llvm.org/D69268

[RISCV] Add support for -ffixed-xX flags
This adds support for reserving GPRs such that the compiler will not 
choose a register for register allocation. The implementation follows 
the same design as for AArch64; each reserved register becomes a target 
feature and used for getting the reserved registers for a given 
MachineFunction. The backend checks that it does not need to write to 
any reserved register; if it does a relevant error is generated.
Differential Revision: https://reviews.llvm.org/D67185

fix a few typos to test git committing

Revert "Fix lld detection in standalone compiler-rt."
Breaks sanitizer-android buildbot.
This reverts commit d56203201f8a1f11abb913c4dfc0bf9c61432d1a.

Fix name of warn_ignored_hip_only_option
Differential Revision: https://reviews.llvm.org/D69268

Revert r374202"[ObjC generics] Fix not inheriting type bounds in
categories/extensions."
This introduced new errors, see below. Reverting until that can be
investigated properly.
  #import <AVFoundation/AVFoundation.h>
  void f(int width, int height) {
   FourCharCode best_fourcc = kCMPixelFormat_422YpCbCr8_yuvs;
   NSDictionary* videoSettingsDictionary = @{
     (id)kCVPixelBufferPixelFormatTypeKey : @(best_fourcc),
   };
 }
  $ clang++ -c /tmp/a.mm
  /tmp/a.mm:6:5: error: cannot initialize a parameter of type
 'KeyType<NSCopying>  _Nonnull const' (aka 'const id') with an rvalue
 of type 'id'
     (id)kCVPixelBufferPixelFormatTypeKey : @(best_fourcc),
     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 1 error generated.
> When a category/extension doesn't repeat a type bound, corresponding
> type parameter is substituted with `id` when used as a type argument.
As
> a result, in the added test case it was causing errors like
>
> > type argument 'T' (aka 'id') does not satisfy the bound
('id<NSCopying>') of type parameter 'T'
>
> We are already checking that type parameters should be consistent
> everywhere (see `checkTypeParamListConsistency`) and update
> `ObjCTypeParamDecl` to have correct underlying type. And when we use
the
> type parameter as a method return type or a method parameter type, it
is
> substituted to the bounded type. But when we use the type parameter as
a
> type argument, we check `ObjCTypeParamType` that ignores the updated
> underlying type and remains `id`.
>
> Fix by desugaring `ObjCTypeParamType` to the underlying type, the same
> way we are doing with `TypedefType`.
>
> rdar://problem/54329242
>
> Reviewers: erik.pilkington, ahatanak
>
> Reviewed By: erik.pilkington
>
> Subscribers: jkorous, dexonsmith, ributzka, cfe-commits
>
> Differential Revision: https://reviews.llvm.org/D66696

typo fix test commit

[PowerPC][NFC] Remove deprecated Function Attrs comments

Fix broken sphinx link in CMake.rst.
Reviewers: delcypher, beanz
Subscribers: mgorny, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69325

[PowerPC][NFC] Remove deprecated Function Attrs comments #2

[NFC] Strip trailing whitespace in test to test Github committing

[libcxx][test][NFC] Fix comment typos.
(Testing git commit access.)

[c++2a] Allow comparison functions to be explicitly defaulted.
This adds some initial syntactic checking that only the appropriate 
function signatures can be defaulted. No implicit definitions are 
generated yet.

[TargetLowering] optimizeSetCCToComparisonWithZero(): add extra sanity
checks (PR43769)
We should do the fold only if both constants are plain, non-opaque
constants, at least that is the DAG.FoldConstantArithmetic() 
requirement. And if the constant we are comparing with is zero - we
shouldn't be trying to do this fold in the first place.
Fixes https://bugs.llvm.org/show_bug.cgi?id=43769

[MIPS GlobalISel] Select MSA vector generic and builtin sub
Select vector G_SUB for MIPS32 with MSA. We have to set bank for vector
operands to fprb and selectImpl will do the rest.
__builtin_msa_subv_<format> will be transformed into G_SUB in
legalizeIntrinsic and selected in the same way.
__builtin_msa_subvi_<format> will be directly selected into 
SUBVI_<format> in legalizeIntrinsic.
Differential Revision: https://reviews.llvm.org/D69306

[MIPS GlobalISel] Select MSA vector generic and builtin mul
Select vector G_MUL for MIPS32 with MSA. We have to set bank for vector
operands to fprb and selectImpl will do the rest. Manual selection of
G_MUL is now done for gprb only.
__builtin_msa_mulv_<format> will be transformed into G_MUL in
legalizeIntrinsic and selected in the same way.
Differential Revision: https://reviews.llvm.org/D69310

[lib/ObjectYAML] - Add a full stop to the comment. NFC.
A test commit.

[DebugInfo] Stop describing imms in TargetInstrInfo's
describeLoadedValue() impl
Summary: The default implementation of the describeLoadedValue() hook
uses the MoveImm property to determine if an instruction moves an
immediate. If an instruction has that property the function returns the
second operand, assuming that that is the immediate value the
instruction moves. As far as I can tell, the MoveImm property does not
imply that the second operand is the immediate value, nor that any other
operand necessarily holds the immediate value; it just means that the 
instruction moves some immediate value.
One example where the second operand is not the immediate is SystemZ's 
LZER instruction, which moves a zero immediate implicitly: $f0S = LZER.
That case triggered an out-of-bound assertion when getting the operand. 
I have added a test case for that instruction.
Another example is ARM's MVN instruction, which holds the logical 
bitwise NOT'd value of the immediate that is moved. For the following 
reproducer:
  extern void foo(int);
 int main() { foo(-11); }
an incorrect call site value would be emitted:
  $ clang --target=arm foo.c -O1 -g -Xclang -femit-debug-entry-values \
     -c -o - | ./build/bin/llvm-dwarfdump  - | \
     grep -A2 call_site_parameter
  0x00000058:       DW_TAG_GNU_call_site_parameter
                     DW_AT_location (DW_OP_reg0 R0)
                     DW_AT_GNU_call_site_value (DW_OP_lit10)
Another example is the A2_combineii instruction on Hexagon which moves 
two immediates to a super-register: $d0 = A2_combineii 20, 10.
Perhaps these are rare exceptions, and most MoveImm instructions hold 
the immediate in the second operand, but in my opinion the default 
implementation of the hook should only describe values that it can, by 
some contract, guarantee are safe to describe, rather than leaving it up 
to the targets to override the exceptions, as that can silently result 
in incorrect call site values.
This patch adds X86's relevant move immediate instructions to the 
target's hook implementation, so this commit should be a NFC for that 
target. We need to do the same for ARM and AArch64.
Reviewers: djtodoro, NikolaPrica, aprantl, vsk
Reviewed By: vsk
Subscribers: kristof.beyls, hiraditya, llvm-commits
Tags: #debug-info, #llvm
Differential Revision: https://reviews.llvm.org/D69109

[Mips] Use appropriate private label prefix based on Mips ABI
MipsMCAsmInfo was using '$' prefix for Mips32 and '.L' for Mips64 
regardless of -target-abi option. By passing MCTargetOptions to
MCAsmInfo we can find out Mips ABI and pick appropriate prefix.
Tags: #llvm, #clang, #lldb
Differential Revision: https://reviews.llvm.org/D66795

[Support] Add a way to run a function on a detached thread
This roughly mimics `std::thread(...).detach()` except it allows to 
customize the stack size. Required for https://reviews.llvm.org/D50993.
I've decided against reusing the existing `llvm_execute_on_thread`
because it's not obvious what to do with the ownership of the passed 
function/arguments:
1. If we pass possibly owning functions data to
`llvm_execute_on_thread`,
  we'll lose the ability to pass small non-owning non-allocating
functions
  for the joining case (as it's used now). Is it important enough? 2. If
we use the non-owning interface in the new use case, we'll force
  clients to transfer ownership to the spawned thread manually, but
  similar code would still have to exist inside
  `llvm_execute_on_thread(_async)` anyway (as we can't just pass the
same
  non-owning pointer to pthreads and Windows implementations, and would
be
  forced to wrap it in some structure, and deal with its ownership.
Patch by Dmitry Kozhevnikov!
Differential Revision: https://reviews.llvm.org/D51103

Revert "[Support] Add a way to run a function on a detached thread"
This reverts commit 40668abca4d307e02b33345cfdb7271549ff48d0. This
causes clang tests to fail, as stacksize=0 is being explicitly passed
and is no longer a no-op.

[Sanitizers] Add support for RISC-V 64-bit
Summary: This has been tested with gcc trunk on openSUSE Tumbleweed on
the HiFive Unleashed.
Patch by Andreas Schwab (schwab)
Reviewers: luismarques
Reviewed By: luismarques
Subscribers: mhorne, emaste, luismarques, asb, mgorny, fedor.sergeev,
simoncook, kito-cheng, shiva0217, rogfer01, rkruppe, lenary, s.egerton,
#sanitizers, llvm-commits
Tags: #llvm, #sanitizers
Differential Revision: https://reviews.llvm.org/D66870

[clangd] Propogate context in TUScheduler::run
Reviewers: sammccall
Subscribers: ilya-biryukov, javed.absar, MaskRay, jkorous, arphaman,
usaxena95, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D69328

Reland "[Support] Add a way to run a function on a detached thread""
This reverts commit 7bc7fe6b789d25d48d6dc71d533a411e9e981237. The
immediate callers have been fixed to pass nullopt where appropriate.

[X86] combineX86ShufflesRecursively - assert the root mask is legal.
NFCI.

[mips] Reformat `la / dla` expansion test cases. NFC

[mips] Add tests to check `la / dla` expansion in XGOT cases. NFC

[mips] Implement `la` macro expansion for N32 ABI

[mips] Use `expandLoadAddress` for JAL expansion
- Reduce code duplication
- Get partial support of JAL expansion for XGOT.

[libc++][NFC] Remove excess trailing newlines from most files
Testing git commit access.

[analyzer] Fix off-by-one in operator call parameter binding.
Member operator declarations and member operator expressions have
different numbering of parameters and arguments respectively: one of
them includes "this", the other does not.
Account for this inconsistency when figuring out whether the parameter
needs to be manually rebound from the Environment to the Store when
entering a stack frame of an operator call, as opposed to being
constructed with a constructor and as such already having the necessary
Store bindings.
Differential Revision: https://reviews.llvm.org/D69155

[LVI][CVP] LazyValueInfoImpl::solveBlockValueBinaryOp(): use no-wrap
flags from `add` op
Summary: This was suggested in https://reviews.llvm.org/D69277#1717210 
In this form (this is what was suggested, right?), the results aren't
staggering
(especially since given LVI cross-block focus) this does catch some
things (as per test-suite), but not too much:
| statistic                                        |       old |      
new | delta | % change |
| correlated-value-propagation.NumAddNSW           |      4981 |     
4982 |     1 |  0.0201% |
| correlated-value-propagation.NumAddNW            |     12125 |    
12126 |     1 |  0.0082% |
| correlated-value-propagation.NumCmps             |      1199 |     
1202 |     3 |  0.2502% |
| correlated-value-propagation.NumDeadCases        |       112 |      
111 |    -1 | -0.8929% |
| correlated-value-propagation.NumMulNSW           |       275 |      
278 |     3 |  1.0909% |
| correlated-value-propagation.NumMulNUW           |      1323 |     
1326 |     3 |  0.2268% |
| correlated-value-propagation.NumMulNW            |      1598 |     
1604 |     6 |  0.3755% |
| correlated-value-propagation.NumNSW              |      7158 |     
7167 |     9 |  0.1257% |
| correlated-value-propagation.NumNUW              |     13304 |    
13310 |     6 |  0.0451% |
| correlated-value-propagation.NumNW               |     20462 |    
20477 |    15 |  0.0733% |
| correlated-value-propagation.NumOverflows        |         4 |        
7 |     3 | 75.0000% |
| correlated-value-propagation.NumPhis             |     15366 |    
15381 |    15 |  0.0976% |
| correlated-value-propagation.NumSExt             |      6273 |     
6277 |     4 |  0.0638% |
| correlated-value-propagation.NumShlNSW           |      1172 |     
1171 |    -1 | -0.0853% |
| correlated-value-propagation.NumShlNUW           |      2793 |     
2794 |     1 |  0.0358% |
| correlated-value-propagation.NumSubNSW           |       730 |      
736 |     6 |  0.8219% |
| correlated-value-propagation.NumSubNUW           |      2044 |     
2046 |     2 |  0.0978% |
| correlated-value-propagation.NumSubNW            |      2774 |     
2782 |     8 |  0.2884% |
| instcount.NumAddInst                             |    277586 |   
277569 |   -17 | -0.0061% |
| instcount.NumAndInst                             |     66056 |    
66054 |    -2 | -0.0030% |
| instcount.NumBrInst                              |    709147 |   
709146 |    -1 | -0.0001% |
| instcount.NumCallInst                            |    528579 |   
528576 |    -3 | -0.0006% |
| instcount.NumExtractValueInst                    |     18307 |    
18301 |    -6 | -0.0328% |
| instcount.NumOrInst                              |    102660 |   
102665 |     5 |  0.0049% |
| instcount.NumPHIInst                             |    318008 |   
318007 |    -1 | -0.0003% |
| instcount.NumSelectInst                          |     46373 |    
46370 |    -3 | -0.0065% |
| instcount.NumSExtInst                            |     79496 |    
79488 |    -8 | -0.0101% |
| instcount.NumShlInst                             |     40654 |    
40657 |     3 |  0.0074% |
| instcount.NumTruncInst                           |     62251 |    
62249 |    -2 | -0.0032% |
| instcount.NumZExtInst                            |     68211 |    
68221 |    10 |  0.0147% |
| instcount.TotalBlocks                            |    843910 |   
843909 |    -1 | -0.0001% |
| instcount.TotalInsts                             |   7387448 |  
7387423 |   -25 | -0.0003% |
Reviewers: nikic, reames
Reviewed By: nikic
Subscribers: hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69321

[LVI][NFC] Factor solveBlockValueSaturatingIntrinsic() out of
solveBlockValueIntrinsic()
Now that there's SaturatingInst class, this is cleaner.

Fix non-portable GNU diff option
Summary: This is a fix to revision D68839 and rL375023. This patch
substitutes POSIX option "-b" for the non-portable GNU option
"--strip-trailing-cr".
Reviewers: daltenty, hubert.reinterpretcast
Reviewed By: daltenty
Subscribers: mehdi_amini, hiraditya, steven_wu, dexonsmith, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69342

[Analysis] Update Analysis/LazyValueAnalysis/lvi-after-jumpthreading.ll
I should have updated it in 1f665046fbf3b9d47a229714f689cd941f6f1216 but
i didn't even realize those tests were there.

[clangd] abort if shutdown takes more than a minute.
Summary: A certain class of bug (e.g. infloop on an AST worker thread)
currently means clangd never terminates, even if the editor shuts down
the protocol and closes our stdin, and the main thread recognizes that.
Instead, let's wait 60 seconds for threads to finish cleanly, and then
crash if they haven't.
(Obviously, we should still fix these bugs).
Reviewers: kadircet
Subscribers: MaskRay, jkorous, arphaman, jfb, usaxena95, cfe-commits,
ilya-biryukov
Tags: #clang
Differential Revision: https://reviews.llvm.org/D69329

Minor movement of one function with now-correct strictfp attribute to
pass against the latest version of D68233.

[lldb] Add nodebug attribute to import-std-module/sysroot test
Summary: So far we rely on the default argument and the fact that we
don't call this inline function in our actual `main.cpp` to make sure
that this function can only be called if LLDB loads this header as a C++
module. This patch just adds the nodebug attribute as yet another
measure to make sure LLDB can't call this function without the standard
module loaded. Note that the test is already requiring clang for the
sysroot setup, so its fine that this is a Clang specific attribute.
Reviewers: friss, labath
Subscribers: JDevlieghere, lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D68861

Revert 4334892e7b "[DAGCombine][ARM] x ==/!= c  ->  (x - c) ==/!= 0  iff
'-c' can be folded into the x node."
This broke various Windows builds, see comments on the Phabricator 
review.
This also reverts the follow-up 20bf0cf.
> Summary:
> This fold, helps recover from the rest of the D62266 ARM regressions.
> https://rise4fun.com/Alive/TvpC
>
> Note that while the fold is quite flexible, i've restricted it
> to the single interesting pattern at the moment.
>
> Reviewers: efriedma, craig.topper, spatel, RKSimon, deadalnix
>
> Reviewed By: deadalnix
>
> Subscribers: javed.absar, kristof.beyls, llvm-commits
>
> Tags: #llvm
>
> Differential Revision: https://reviews.llvm.org/D62450

[NFC] Strip trailing whitespace from libc++

Remove a no longer accurate sentence from the coding standards.
(And test my commit access. We're working on larger changes here.)

[libcxx][NFC] Strip trailing whitespace, fix typo.

Improve language in GettingStarted.rst
This patch was reviewed and approved by chandlerc.

"Getting Started with the LLVM System" is the first point of contact for
many newcomers in the LLVM community.
* Make the first two paragraphs more welcoming
* Use more inclusive language

Use portable flag with nm in extract_symbols.py
Summary: nm is one of the tools that extract_symbols.py can use to
extract symbols from llvm libraries as part of the build process. This
patch updates the invocation of nm to use the -P POSIX option for
"portable output" so we get a consistently parsable output format on all 
platforms.
A link to the relevant nm format:
https://pubs.opengroup.org/onlinepubs/9699919799/utilities/nm.html
Reviewers: hubert.reinterpretcast, stevewan, sfertile
Reviewed By: stevewan
Subscribers: llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69004

Improve Clang's getting involved document and make it more inclusive in
wording.
Summary: Working with Meike and others to improve the wording in this
document.
Reviewers: klimek
Subscribers: mcrosier, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D69351

Add beginning of LLVM's GettingStarted to GitHub readme
Reviewed and approved by chandlerc.
 As GitHub is the canonical LLVM repository now, embrace GitHub's way of
displaying basic build instructions in the top-level readme.md.

[Hexagon] Fix typo. NFC
Testing git push access.

[AMDGPU] Allow folding of sgpr to vgpr copy
Potentially sgpr to sgpr copy should also be possible. That is however
trickier because we may end up with a wrong register class at use
because of xm0/xexec permutations.
Differential Revision: https://reviews.llvm.org/D69280

[X86] Replace some regular expressions in xray tests with explicit
checks to show bad assembly.
We're print 16-bit or 32-bit registers in copy instructions to 64-bit
registers. This code will not assemble if it were to be parsed back in.
Emitting to binary works because we'll encode the register the same way
no matter what the size is.

[MIPS GlobalISel] MSA vector generic and builtin sdiv, srem, udiv, urem
Select vector G_SDIV, G_SREM, G_UDIV and G_UREM for MIPS32 with MSA. We 
have to set bank for vector operands to fprb and selectImpl will do the 
rest. __builtin_msa_div_s_<format>, __builtin_msa_mod_s_<format>,
__builtin_msa_div_u_<format> and __builtin_msa_mod_u_<format> will be 
transformed into G_SDIV, G_SREM, G_UDIV and G_UREM in legalizeIntrinsic 
respectively and selected in the same way.
Differential Revision: https://reviews.llvm.org/D69333

[MIPS GlobalISel] MSA vector generic and builtin fadd, fsub, fmul, fdiv
Select vector G_FADD, G_FSUB, G_FMUL and G_FDIV for MIPS32 with MSA. We 
have to set bank for vector operands to fprb and selectImpl will do the 
rest. __builtin_msa_fadd_<format>, __builtin_msa_fsub_<format>,
__builtin_msa_fmul_<format> and __builtin_msa_fdiv_<format> will be 
transformed into G_FADD, G_FSUB, G_FMUL and G_FDIV in legalizeIntrinsic 
respectively and selected in the same way.
Differential Revision: https://reviews.llvm.org/D69340

[clangd] Collect name references in the index.
Summary: This is used for cross-file rename. When renaming a class, we
expect to rename all related constructors/destructors.
Reviewers: kadircet, ilya-biryukov
Subscribers: MaskRay, jkorous, arphaman, usaxena95, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D69338

[clangd] Handle the missing constructor initializers in
findExplicitReferences.
Reviewers: ilya-biryukov
Subscribers: MaskRay, jkorous, arphaman, kadircet, usaxena95,
cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D69241

Hide implementation details in anonymous namespaces. NFC.

[libFuzzer] docs: update note to include REDUCE event.

Expose __hwasan_tag_mismatch_stub
Summary: GCC would like to emit a function call to report a tag mismatch 
rather than hard-code the `brk` instruction directly.
__hwasan_tag_mismatch_stub contains most of the functionality to do this
already, but requires exposure in the dynamic library.
This patch moves __hwasan_tag_mismatch_stub outside of the anonymous 
namespace that it was defined in and declares it in 
hwasan_interface_internal.h.
We also add the ability to pass sizes larger than 16 bytes to this 
reporting function by providing a fourth parameter that is only looked 
at when the size provided is not in the original accepted range.
This does not change the behaviour where it is already being called, 
since the previous definition only accepted sizes up to 16 bytes and 
hence the change in behaviour is not seen by existing users. The change
in declaration does not matter, since the only existing use is in the
__hwasan_tag_mismatch function written in assembly.
Tested with gcc and clang on an AArch64 vm.
Reviewers: eugenis, kcc, pcc, #sanitizers
Reviewed By: eugenis, #sanitizers
Subscribers: kristof.beyls, llvm-commits
Tags: #sanitizers, #llvm
Differential Revision: https://reviews.llvm.org/D69113

Revert "Expose __hwasan_tag_mismatch_stub"
Attribution to author of patch got lost.
This reverts commit 612eadb7bc06b8f1a094976e06155f46ebd70d7c.

[compiler-rt] Expose __hwasan_tag_mismatch_stub
Summary: GCC would like to emit a function call to report a tag mismatch 
rather than hard-code the `brk` instruction directly.
__hwasan_tag_mismatch_stub contains most of the functionality to do this
already, but requires exposure in the dynamic library.
This patch moves __hwasan_tag_mismatch_stub outside of the anonymous 
namespace that it was defined in and declares it in 
hwasan_interface_internal.h.
We also add the ability to pass sizes larger than 16 bytes to this 
reporting function by providing a fourth parameter that is only looked 
at when the size provided is not in the original accepted range.
This does not change the behaviour where it is already being called, 
since the previous definition only accepted sizes up to 16 bytes and 
hence the change in behaviour is not seen by existing users. The change
in declaration does not matter, since the only existing use is in the
__hwasan_tag_mismatch function written in assembly.
Reviewers: eugenis, kcc, pcc, #sanitizers
Reviewed By: eugenis, #sanitizers
Subscribers: kristof.beyls, llvm-commits
Tags: #sanitizers, #llvm
Differential Revision: https://reviews.llvm.org/D69113
Patch by Matthew Malcomson <matthew.malcomson@arm.com>

Don't add -fsplit-lto-unit for thin LTO builds with PS4 and Darwin
toolchains
These toolchains use legacy thin LTO API, which is not capable of unit
splitting Differential revision: https://reviews.llvm.org/D69173

[MIPS GlobalISel] Select MSA vector generic and builtin fabs
selectImpl is able to select G_FABS when we set bank for vector operands
to fprb. Add detailed tests. Note: G_FABS is generated from llvm-ir
intrinsics llvm.fabs.*, and at the moment MIPS is not able to generate
this intrinsic for vector type (some targets generate vector llvm.fabs.*
from calls to a builtin function). We can handle fabs using
__builtin_msa_fmax_a_<format> and passing same vector as both arguments.
__builtin_msa_fmax_a_<format> will be directly selected into
FMAX_A_<format> in legalizeIntrinsic.
Differential Revision: https://reviews.llvm.org/D69346