Changes

[lldb] Add lldb reproducer test stage

Fix missing '\' in pipeline

[SVE] Pass Scalable argument to VectorType::get in Bitcode Reader
Pass the Scalability test to VectorType::get in order to be able to 
deserialize bitcode that contains scalable vector operations
Differential Revision: https://reviews.llvm.org/D73144

[x86] fold vperm2x128 to concat of 128-bit high half vectors
vperm (ins ?, X, C), (ins ?, Y, C), 0x31 --> concat X, Y
This is another shuffle problem seen with PR42024: 
https://bugs.llvm.org/show_bug.cgi?id=42024
We have this small crack in legalization/lowering/combining/demanded 
that allows forming a vperm2f128 of high halves with AVX1 when we could
do better by peeking through the insert_subvector nodes. AFAICT, it
requires IR as shown in the diffs - much larger than legal vectors - to
avoid all of the usual folds.
Another option would prevent forming the 256-bit vperm in lowering.
Differential Revision: https://reviews.llvm.org/D73197

[llvm-cov] Add support for -skip-functions to lcov
Summary: This flag was added for the json format to exclude functions
from the output. This mirrors that behavior in lcov (where it was
previously accepted but ignored). This makes the output file smaller
which can be beneficial depending on how you consume it, especially if
you don't use this data anyways.
Patch by Keith Smiley (@keith).
Reviewers: kastiglione, Dor1s, vsk, allevato
Reviewed By: Dor1s, allevato
Subscribers: llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D73160

[libFuzzer] Add INFO output when LLVMFuzzerCustomMutator is found.
Summary: there is an ongoing work on interchangeable custom mutators
(https://github.com/google/clusterfuzz/pull/1333/files#r367706283) and
having some sort of signalling from libFuzzer that it has loaded a
custom mutator would be helpful.
The initial idea was to make the mutator to print something, but given 
the anticipated variety of different mutators, it does not seem possible 
to make all of them print the same message to signal their execution.
Reviewers: kcc, metzman
Reviewed By: metzman
Subscribers: #sanitizers, llvm-commits
Tags: #sanitizers, #llvm
Differential Revision: https://reviews.llvm.org/D73136

R600: Fix failing testcase

Convert AssertTrue( A == B) to AssertEqual(A, B) in TestObjCStepping.py.

[lldb/Util] Fix lldb-repro now it doesn't take a path to lldb
The indices into the arguments array were off because we no longer pass 
the path to lldb as the first argument.

[lldb/Test] Use lit's capabilities to skip lldb-repro tests.
This allows us to skip the reproducer tests themselves as a whole as 
well as individual tests with the UNSUPPORTED keyword.

[lldb/Reproducer] Correctly instrument enum values
Enum values can be serialized trivially and should not go through the 
object registry.

[ODRHash] Factor out functionality for CXXRecord ODR diagnostics (NFCI)
There's going to be a lot of common code between RecordDecl and 
CXXRecordDecl, factor out some of the logic in preparation for adding
the RecordDecl side.

[PatternMatch] Add m_APInt/m_APFloat matchers accepting undef
The current m_APInt() and m_APFloat() matchers do not accept splats that
include undefs (unlike m_Zero() and other matchers for specific values).
We can't simply change the default behavior, as there are existing
transforms that would not be safe with undefs.
For this reason, I'm introducing new m_APIntAllowUndef() and 
m_APFloatAllowUndef() matchers, that allow splats with undefs. 
Additionally, m_APIntForbidUndef() and m_APFloatForbidUndef() are added.
These have the same behavior as the existing m_APInt() and m_APFloat(),
but serve as an explicit indication that undefs were considered and
found unsound for this transform. This helps distinguish them from
existing uses of m_APInt() where we do not know whether undefs can or
cannot be allowed without additional review.
Differential Revision: https://reviews.llvm.org/D72975

Clang] Fix expansion of response files in -Wp after integrated-cc1
change
After rGb4a99a061f517e60985667e39519f60186cbb469, passing a response
file such as -Wp,@a.rsp wasn't working anymore because .rsp expansion
happens inside clang's main() function.
This patch adds response file expansion in the -cc1 tool.
Differential Revision: https://reviews.llvm.org/D73120

[PowerPC] Collect some CallLowering arguments into a struct. [NFC]
Collect the calling convention and a number of boolean arguments into a 
structure to slightly reduces the number of arguments passed around
between LowerCall_<Subtarget>, FinishCall and a few of the helpers. Also 
calulates if a call is indirect once using the exisitng helper and
caches the result replacing several instances where we duplicated the
logic determining if a call is indirect.

[PatternMatch] Make m_c_ICmp swap the predicate (PR42801)
This addresses https://bugs.llvm.org/show_bug.cgi?id=42801. The
m_c_ICmp() matcher is changed to provide the swapped predicate if the
operands are swapped.
Existing uses of m_c_ICmp() fall in one of two categories: Working on
equality predicates only, where swapping is irrelevant. Or performing a
manual swap, in which case this patch removes it.
The only exception is the foldICmpWithLowBitMaskedVal() fold, which does
not swap the predicate, and instead reasons about whether a swap
occurred or not for each predicate. Getting the swapped predicate allows
us to merge the logic for pairs of predicates, instead of duplicating
it.
Differential Revision: https://reviews.llvm.org/D72976

[TestStdModuleSysroot] Only run locally.

[mlir][Linalg] Add tensor support to Linalg EDSC Builders
Summary: This diff extends the Linalg EDSC builders so we can easily
create mixed tensor/buffer linalg.generic ops. This is expected to be
useful for HLO -> Linalg lowering.
The StructuredIndexed struct is made to derive from ValueHandle and can 
now capture a type + indexing expressions. This is used to represent
return tensors.
Pointwise unary and binary builders are extended to allow both output
buffers and return tensors. This has implications on the number of
region arguments.
Reviewers: ftynse, hanchung, asaadaldien
Subscribers: mehdi_amini, rriddle, jpienaar, burmako, shauheen,
antiagainst, arpith-jacob, mgester, lucyrfox, aartbik, liufengdb,
llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D73149

[X86] Cast to __v4hi instead of __m64 in the implementation of
_mm_extract_pi16 and _mm_insert_pi16.
__m64 is a vector of 1 long long. But the builtins these intrinsics are
calling expect a vector of 4 shorts.
Fixes PR44589

[InstCombine] Add test for PR44529; NFC

[InstCombine] Combine neg of shl of sub (PR44529)
Fixes https://bugs.llvm.org/show_bug.cgi?id=44529. We already have a
combine to sink a negation through a left-shift, but it currently only
works if the shift operand is negatable without creating any 
instructions. This patch introduces freelyNegateValue() as a more 
powerful extension of dyn_castNegVal(), which allows negating a value as
long as this doesn't end up increasing instruction count. Specifically,
this patch adds support for negating A-B to B-A.
This mechanism could in the future be extended to handle general 
negation chains that a) start at a proper 0-X negation and b) only 
require one operand to be freely negatable. This would end up as a 
weaker form of D68408 aimed at the most obviously profitable subset that
eliminates a negation entirely.
Differential Revision: https://reviews.llvm.org/D72978

[www] Update cxx_status page to list changes on the Clang 10 branch as
"Clang 10" not "SVN".

[lldb/Utility] Don't forward directories to the file collector
The VFS mapping writer assumes that all the paths it gets are files. 
When passed a directory, it ends up as a file in the VFS mapping twice, 
once as a file and once as a directory.
  {
   'type': 'file',
   'name': "Output",
   'external-contents': "/root/path/to/Output"
 },
 {
   'type': 'directory',
   'name': "Output",
   'contents': [ ... ]
 }

[lldb/Test] Skip script interpreter tests reading from stdin for
lldb-repro
The reproducers currently only shadow the command interpreter. It would 
be possible to make it work for the Lua interpreter which uses the 
IOHandlerEditline under the hood, but the Python one runs a REPL in 
Python itself so there's no (straightforward) way to shadow that.
Given that we already capture any API calls, this isn't super high on my 
list of priorities.

[LoopIdiomRecognize] Reduce variable scope. [NFCI]

[IndVarSimplify] Cleanup spaces and reduce variable scope [NFCI]
Minor clean-ups + clang-format.

Embed a zero-length /dev/null in darwin-debug for the special section.
Fred suggested that instead of embedded CMakeLists.txt in the binary as 
the contents of a special section, see if /dev/null would work.  It 
does.

[nfc][libomptarget] Remove SHARED annotation from local variables
Summary:
[nfc][libomptarget] Remove SHARED annotation from local variables
A few local variables in reduction.cu were marked SHARED. This patch
leaves all per-kernel global state localised in omp_data.cu.
Reviewers: ABataev, jdoerfert, grokos
Reviewed By: jdoerfert
Subscribers: openmp-commits
Tags: #openmp
Differential Revision: https://reviews.llvm.org/D73239

[cmake] Fix clang builds with BUILD_SHARED=ON and
CLANG_LINK_CLANG_DYLIB=ON
Summary: We were linking all the clang objects and shared libraries into 
libclang-cpp.so, which was causing the command line options to be 
registered twice.
Reviewers: beanz, mgorny
Reviewed By: beanz, mgorny
Subscribers: mgorny, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D68520

Revert "[cmake] Fix clang builds with BUILD_SHARED=ON and
CLANG_LINK_CLANG_DYLIB=ON"
This reverts commit df839cfda09dbadc26b8be635f27da75f1f27190.
This change used cmake's list filter operation which was not added until
cmake 3.6.

[lldb/Reproducer] Disable buffering of stdout during replay
Different buffering behavior during capture and replay caused some of 
the shell tests to fail when run from a reproducer. By disabling stdout 
buffering we get a better approximation of how things get flushed during 
an regular debug session. There is a performance impact but since this 
only affects replay this is acceptable.

[IndVarSimplify] Teach IndVarSimplify to preserve MemorySSA.

[AArch64TTI] AArch64 supports NT vector stores through STNP.
This patch adds a custom implementation of isLegalNTStore to AArch64TTI 
that supports vector types that can be directly stored by STNP. Note 
that the implementation may not catch all valid cases (e.g. because the 
vector is a multiple of 256 and could be broken down to multiple valid
256 bit stores), but it is good enough for LV to vectorize loops with NT
stores, as LV only passes in a vector with 2 elements to check. LV seems
to also be the only user of isLegalNTStore.
We should also do the same for NT loads, but before that we need to 
ensure that we properly lower LDNP of vectors, similar to D72919.
Reviewers: dmgreen, samparker, t.p.northover, ab
Reviewed By: dmgreen
Differential Revision: https://reviews.llvm.org/D73158

[lldb/Reproducer] Mark some driver tests  as unsupported for lldb-repro
These test are checking for diagnostics printed by the driver. During 
replay we only replay the SB API calls made by the driver, so it's 
expected that these messages aren't displayed.

Replace old-style cast of null pointer with nullptr

[lldb/Test] Add check-lldb-repro target
This adds a new target check-lldb-repro which runs the shell tests with 
the lldb-repo utility. It runs the shell tests twice, once while 
capturing a reproducer and then again by replaying that reproducer.

[lldb/Test] Fix type in add_lit_testsuite
The new test suite should be called check-lldb-repro rather than 
check-lldb.

[lldb/Docs] Document testing strategies for the reproducers
Document the different ways we test the reproducers. This is mostly to 
describe the new check-lldb-repro target.

[RISCV] Fix evaluating %pcrel_lo against global and weak symbols
Summary: Previously, we would erroneously turn %pcrel_lo(label), where
label has a %pcrel_hi against a weak symbol, into %pcrel_lo(label +
offset), as evaluatePCRelLo would believe the target independent logic
was going to fold it. Moreover, even if that were fixed,
shouldForceRelocation lacks an MCAsmLayout and thus cannot evaluate the
%pcrel_hi fixup to a value and check the symbol, so we would then
erroneously constant-fold the
%pcrel_lo whilst leaving the %pcrel_hi intact. After D72197, this same 
sequence also occurs for symbols with global binding, which is triggered 
in real-world code.
Instead, as discussed in D71978, we introduce a new FKF_IsTarget flag to 
avoid these kinds of issues. All the resolution logic happens in one 
place, with no coordination required between RISCAsmBackend and 
RISCVMCExpr to ensure they implement the same logic twice. Although the 
implementation of %pcrel_hi can be left as target independent, we make 
it target dependent to ensure that they are handled identically to
%pcrel_lo, otherwise we risk one of them being constant folded but the 
other being preserved. This also allows us to properly support fixup 
pairs where the instructions are in different fragments.
Reviewers: asb, lenary, efriedma
Reviewed By: efriedma
Subscribers: arichardson, hiraditya, rbar, johnrusso, simoncook,
sabuasal, niosHD, kito-cheng, shiva0217, MaskRay, zzheng, edward-jones,
rogfer01, MartinMosbeck, brucehoult, the_o, rkruppe, PkmX, jocewei,
psnobl, benna, Jim, s.egerton, pzheng, sameer.abuasal, apazos,
luismarques, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D73211

[llvm/Transforms] Fix warning: private field 'MSSA' is not used

[lldb] Add lldb reproducer test stage

Fix missing '\' in pipeline