Changes

[LNT] Update git URL in Dockerfile
Summary: Update LNT git repo URL in Dockerfile from the old git mirror
to the current github official repo. This will allow the Docker image to 
actually work as a Python 3 service.
Reviewers: cmatthews, hubert.reinterpretcast, kristof.beyls,
PrzemekWirkus, leandron
Reviewed By: leandron
Subscribers: llvm-commits, MatzeB, leandron, PrzemekWirkus
Differential Revision: https://reviews.llvm.org/D73611

[NFC][IndVarSimplify] Autogenerate tests affected by
isHighCostExpansionHelper() cost modelling (PR44668)

[NFC][LoopVectorize] Autogenerate tests affected by
isHighCostExpansionHelper() cost modelling (PR44668)

Fix an assertion failure in DwarfExpression's subregister composition
This patch fixes an assertion failure in DwarfExpression that is 
triggered when a complex fragment has exactly the size of a subregister
of the register the DBG_VALUE points to *and* there is no DWARF encoding
for the super-register.
I took the opportunity to replace/document some magic values with static
constructor functions to make this code less confusing to read.
rdar://problem/58489125
Differential Revision: https://reviews.llvm.org/D72938

[DAG] Enable ISD::EXTRACT_SUBVECTOR SimplifyMultipleUseDemandedBits
handling
This allows SimplifyDemandedBits to call SimplifyMultipleUseDemandedBits
to create a simpler ISD::EXTRACT_SUBVECTOR, which is particularly useful
for cases where we're splitting into subvectors anyhow.
Differential Revision: This allows SimplifyDemandedBits to call
SimplifyMultipleUseDemandedBits to create a simpler
ISD::EXTRACT_SUBVECTOR, which is particularly useful for cases where
we're splitting into subvectors anyhow.

[mlir] add lowering from affine.min to std
Summary: Affine minimum computation will be used in tiling
transformation. The implementation is mostly boilerplate as we already
lower the minimum in the upper bound of an affine loop.
Differential Revision: https://reviews.llvm.org/D73488

[NFC][mlir][linalg] Merge Utils/Intrinsics.h into EDSC/Intrinsics.h
Differential Revision: https://reviews.llvm.org/D73377

[InstCombine] allow more narrowing of casted select
D47163 created a rule that we should not change the casted type of a
select when we have matching types in its compare condition. That was
intended to help vector codegen, but it also could create situations
where we miss subsequent folds as shown in PR44545: 
https://bugs.llvm.org/show_bug.cgi?id=44545
By using shouldChangeType(), we can continue to get the vector folds
(because we always return false for vector types). But we also solve the
motivating bug because it's ok to narrow the scalar select in that 
example.
Our canonicalization rules around select are a mess, but AFAICT, this 
will not induce any infinite looping from the reverse transform (but 
we'll need to watch for that possibility if committed).
Side note: there's a similar use of shouldChangeType() for phi ops just
below this diff, and the source and destination types appear to be
reversed.
Differential Revision: https://reviews.llvm.org/D72733

[analyzer] Add FuchsiaLockChecker and C11LockChecker
These are mostly trivial additions as both of them are reusing existing 
PThreadLockChecker logic. I only needed to add the list of functions to 
check and do some plumbing to make sure that we display the right 
checker name in the diagnostic.
Differential Revision: https://reviews.llvm.org/D73376

[examples] Fix CMakefiles for JITLink and OrcError library refactoring
The examples need explicit library dependencies when building with 
BUILD_SHARED_LIBS=on

[mlir] Remove 'valuesToRemoveIfDead' from PatternRewriter API
Summary: Remove 'valuesToRemoveIfDead' from PatternRewriter API. The
removal functionality wasn't implemented and we decided [1] not to
implement it in favor of having more powerful DCE approaches.
[1] https://github.com/tensorflow/mlir/pull/212
Reviewers: rriddle, bondhugula
Reviewed By: rriddle
Subscribers: liufengdb, mehdi_amini, rriddle, jpienaar, burmako,
shauheen, antiagainst, nicolasvasilache, arpith-jacob, mgester,
lucyrfox, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D72545

[GlobalISel] Make use of KnownBits::computeForAddSub
Summary: This is mostly NFC. computeForAddSub may give more precise
results in some cases, but that doesn't seem to affect any existing
GlobalISel tests.
Subscribers: rovka, hiraditya, volkan, Petar.Avramovic, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D73431

[gwp-asan] Implement malloc_iterate.
Summary: This is an Android-specific interface for iterating over all
live allocations in a memory range.
Reviewers: hctim, cferris
Subscribers: mgorny, mgrang, #sanitizers, llvm-commits
Tags: #sanitizers, #llvm
Differential Revision: https://reviews.llvm.org/D73305

AMDGPU/GlobalISel: Eliminate SelectVOP3Mods_f32
Trivial type predicates should be moved into the tablegen pattern 
itself, and not checked inside complex patterns. This eliminates a 
redundant complex pattern, and fixes select source modifiers for 
GlobalISel.
I have further patches which fully handle select in tablegen and remove
all of the C++ selection, although it requires the ugliness to support
the entire range of legal register types.

remove a trailing space character (test commit)

[lldb/Test] Use lit.local.cfg to mark whole directory as (un)supported.
Mark the whole Python or Lua test directory as unsupported when the 
corresponding language is not available.

[lldb/Commands] Make column available through _regexp-break
Update _regexp-break to interpret main.c:8:21 as:
  breakpoint set --line 8 --column 21
Differential revision: https://reviews.llvm.org/D73314

[AArch64] Add option to enable/disable load-store renaming.
This patch adds a new option to enable/disable register renaming in the 
load-store optimizer. Defaults to disabled, as there is a potential 
mis-compile caused by this.

Fix StackSafetyAnalysis crash with scalable vector types.
Summary: Treat scalable allocas as if they have storage size of 0, and 
scalable-typed memory accesses as if their range is unlimited.
This is not a proper support of scalable vector types in the analysis - 
we can do better, but not today.
Reviewers: vitalybuka
Subscribers: hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D73394

Support zero size types in StackSafetyAnalysis.
Reviewers: vitalybuka
Subscribers: hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D73395

[WinEH] Re-run stack coloring test for i686
This would've caught https://crbug.com/1045650, which resulted in the 
revert of 7a8b0b1595e7dc878b48cf9bbaa652087a6895db.

[IR] Keep a double break between functions when printing a module
This behavior appears to have changed unintentionally in 
b0e979724f2679e4e6f5b824144ea89289bd6d56.
Instead of printing the leading newline in printFunction, print it when 
printing a module. This ensures that `OS << *Func` starts printing 
immediately on the current line, but whole modules are printed nicely.
Reviewed By: MaskRay
Differential Revision: https://reviews.llvm.org/D73505

[clang-tidy] readability-identifier-naming disregards parameters
restrictions on main like functions
Summary: Typically most main functions have the signature:
``` int main(int argc, char *argv[])
``` To stick with convention when renaming parameters we should ignore
the `argc` and `argv` names even if the parameter style says they should
be renamed. This patch addresses this by checking all ParmVarDecls if
they form part of a function with a signature that matches main `int
name(int argc, char * argv[], (optional char *env[]))`
Reviewers: aaron.ballman, JonasToth, alexfh, hokein
Reviewed By: aaron.ballman
Subscribers: Mordante, merge_guards_bot, xazax.hun, kristof.beyls,
cfe-commits
Tags: #clang, #clang-tools-extra
Differential Revision: https://reviews.llvm.org/D73098

[clang-tidy] readability-redundant-string-init now flags redundant
initialisation in Field Decls and Constructor Initialisers
Summary: The original behaviour of this check only looked at VarDecls
with strings that had an empty string initializer. This has been
improved to check for FieldDecls with an in class initializer as well as
constructor initializers.
Addresses [[ https://bugs.llvm.org/show_bug.cgi?id=44474 | clang-tidy
"modernize-use-default-member-init"/"readability-redundant-string-init"
and redundant initializer of std::string ]]
Reviewers: aaron.ballman, alexfh, hokein
Reviewed By: aaron.ballman
Subscribers: merge_guards_bot, mgorny, Eugene.Zelenko, xazax.hun,
cfe-commits
Tags: #clang, #clang-tools-extra
Differential Revision: https://reviews.llvm.org/D72448

Reland "[StackColoring] Remap PseudoSourceValue frame indices via
MachineFunction::getPSVManager()""
Reland 7a8b0b1595e7dc878b48cf9bbaa652087a6895db, with a fix that checks
`!E.value().empty()` to avoid inserting a zero to SlotRemap.
Debugged by rnk@ in
https://bugs.chromium.org/p/chromium/issues/detail?id=1045650#c33
Reviewed By: rnk
Differential Revision: https://reviews.llvm.org/D73510

[clang-tidy] Fix bugprone-use-after-move when move is in noexcept
operator
Summary: Fixes [[ https://bugs.llvm.org/show_bug.cgi?id=44667 | noexcept
operator misinterpreted as being evaluated ]].
Reviewers: aaron.ballman, alexfh, JonasToth, hokein, gribozavr2
Reviewed By: gribozavr2
Subscribers: merge_guards_bot, Quuxplusone, xazax.hun, cfe-commits
Tags: #clang, #clang-tools-extra
Differential Revision: https://reviews.llvm.org/D73441

[Matrix] Add optimization remarks for matrix expression.
Generate remarks for matrix operations in a function. To generate
remarks for matrix expressions, the following approach is used: 1.
Collect leafs of matrix expressions (done in
  RemarkGenerator::getExpressionLeafs).  Leafs are lowered matrix
  instructions without other matrix users (like stores).
2. For each leaf, create a remark containing a linearizied version of
the
  matrix expression.
The following improvements will be submitted as follow-ups:
* Summarize number of vector instructions generated for each expression.
* Account for shared sub-expressions.
* Propagate matrix remarks up the inlining chain.
The information provided by the matrix remarks helps users to spot cases 
where matrix expression got split up, e.g. due to inlining not 
happening. The remarks allow users to address those issues, ensuring 
best performance.
Reviewers: anemet, Gerolf, thegameg, hfinkel, andrew.w.kaylor, LuoYuanke
Reviewed By: anemet
Differential Revision: https://reviews.llvm.org/D72453

[LV] Remove nondeterminacy by changing
LoopVectorizationLegality::Reductions from DenseMap to MapVector
The iteration order of LoopVectorizationLegality::Reductions matters for
the final code generation, so we better use MapVector instead of
DenseMap for it to remove the nondeterminacy. reduction-order.ll in the
patch is an example reduced from the case we saw. In the output of opt
command, the order of the select instructions in the vector.body block
keeps changing from run to run currently.
Differential Revision: https://reviews.llvm.org/D73490

[runtimes] Fix installation for LLVM_RUNTIME_DISTRIBUTION_COMPONENTS
The installation target we create should trigger the corresponding 
installation target in the runtimes external project.
Differential Revision: https://reviews.llvm.org/D73251

[runtimes] Support install-*-stripped targets
This is needed to support including runtime targets in 
LLVM_DISTRIBUTION_COMPONENTS.
Differential Revision: https://reviews.llvm.org/D73252

[llvm] Fix file ignoring inside directories
We have some ! patterns in the .gitignore (for the projects and runtimes 
directories), and those patterns end up overriding the previous file 
ignores, such that e.g. a .swp file inside the runtimes directory isn't 
ignored. Move the file ignores last to ensure they take effect.
Differential Revision: https://reviews.llvm.org/D73253

[Matrix] Add info about number of operations to remarks.
This patch updates the remark to also include a summary of the number of 
vector operations generated for each matrix expression.
Reviewers: anemet, Gerolf, thegameg, hfinkel, andrew.w.kaylor, LuoYuanke
Reviewed By: anemet
Differential Revision: https://reviews.llvm.org/D72480

[Matrix] Mark remarks test as AArch64 specific.

PR44684: Look through parens and similar constructs when determining
whether a call is to a builtin.
We already had a general mechanism to do this but for some reason 
weren't using it. In passing, check for the other unary operators that 
can intervene in a reasonably-direct function call (we already handled
'&' but missed '*' and '+').

Make AST reading work better with LLVM_APPEND_VC_REV=NO
With LLVM_APPEND_VC_REV=NO,
Modules/merge-lifetime-extended-temporary.cpp would fail if it ran
before a0f50d731639350c7a7 (which changed the serialization format) and
then after, for these reasons:
1. With LLVM_APPEND_VC_REV=NO, the module hash before and after the
  change was the same.
2. Modules/merge-lifetime-extended-temporary.cpp is the only test
  we have that uses -fmodule-cache-path=%t that
  a) actually writes to the cache path
  b) doesn't do `rm -rf %t` at the top of the test
So the old run would write a module file, and then the new run would try
to load it, but the serialized format changed.
Do several things to fix this:
1. Include clang::serialization::VERSION_MAJOR/VERSION_MINOR in
  the module hash, so that when the AST format changes (...and
  we remember to bump these), we use a different module cache dir. 2.
Bump VERSION_MAJOR, since a0f50d731639350c7a7 changed the
  on-disk format in a way that a gch file written before that change
  can't be read after that change. 3. Add `rm -rf %t` to all tests that
pass -fmodule-cache-path=%t.
  This is unnecessary from a correctness PoV after 1 and 2,
  but makes it so that we don't amass many cache dirs over time.
  (Arguably, it also makes it so that the test suite doesn't catch
  when we change the serialization format but don't bump
  clang::serialization::VERSION_MAJOR/VERSION_MINOR; oh well.)
Differential Revision: https://reviews.llvm.org/D73202

[mlir] Add support for marking 'unknown' operations as dynamically
legal.
Summary: This allows for providing a default "catchall" legality check
that is not dependent on specific operations or dialects. For example,
this can be useful to check legality based on the specific types of
operation operands or results.
Differential Revision: https://reviews.llvm.org/D73379

[mlir][NFC] Update Operation::getResultTypes to use ArrayRef<Type>
instead of iterator_range.
Summary: The new internal representation of operation results now allows
for accessing the result types to be more efficient. Changing the API to
ArrayRef is more efficient and removes the need to explicitly
materialize vectors in several places.
Differential Revision: https://reviews.llvm.org/D73429

[mlir] Refactor the implementation of Symbol use lists.
Summary: This revision refactors the implementation of the symbol
use-list functionality to be a bit cleaner, as well as easier to reason
about. Aside from code cleanup, this revision updates the user contract
to never recurse into operations if they define a symbol table. The
current functionality, which does recurse, makes it difficult to examine
the uses held by a symbol table itself. Moving forward users may provide
a specific region to examine for uses instead.
Differential Revision: https://reviews.llvm.org/D73427

[mlir] Add a DCE pass for dead symbols.
Summary: This pass deletes all symbols that are found to be unreachable.
This is done by computing the set of operations that are known to be
live, propagating that liveness to other symbols, and then deleting all
symbols that are not within this live set.
Differential Revision: https://reviews.llvm.org/D72482

[lsan] Support LeakSanitizer runtime on Fuchsia
Support LeakSanitizer runtime on Fuchsia.
Patch By: mcgrathr
Differential Revision: https://reviews.llvm.org/D72887

[Clang] Enable -fsanitize=leak on Fuchsia targets
This required some fixes to the generic code for two issues:
1. -fsanitize=safe-stack is default on x86_64-fuchsia and is *not*
incompatible with -fsanitize=leak on Fuchisa 2. -fsanitize=leak and
other static-only runtimes must not be omitted under -shared-libsan
(which is the default on Fuchsia)
Patch By: mcgrathr
Differential Revision: https://reviews.llvm.org/D73397

[mlir] Replace toy::DeadFunctionEliminationPass with symbolDCEPass.
Summary: The dead function elimination pass in toy was a temporary
stopgap until we had proper dead function elimination support in MLIR.
Now that this functionality is available, this pass is no longer
necessary.
Differential Revision: https://reviews.llvm.org/D72483

Revert "[lsan] Support LeakSanitizer runtime on Fuchsia"
This reverts commit d59e3429f3a6291416e7eb99184448dfe685b77a.

[instrinsics] Add @llvm.memcpy.inline instrinsics
Summary: This is a follow up on D61634. It adds an LLVM IR intrinsic to
allow better implementation of memcpy from C++. A follow up CL will add
the intrinsics in Clang.
Reviewers: courbet, theraven, t.p.northover, jdoerfert, tejohnson
Subscribers: hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D71710

[VE] enable unaligned load/store isel
Summary: Enable unaligned load/store isel for iN and fp32/64 and tests.
Reviewed By: arsenm
Differential Revision: https://reviews.llvm.org/D73448

[Clang] Warn about 'z' printf modifier in old MSVC.
Summary: The 'z' length modifier, signalling that an integer format
specifier takes a `size_t` sized integer, is only supported by the C
library of MSVC 2015 and later. Earlier versions don't recognize the 'z'
at all, and respond to `printf("%zu", x)` by just printing "zu".
So, if the MS compatibility version is set to a value earlier than 
MSVC2015, it's useful to warn about 'z' modifiers in printf format 
strings we check.
Reviewers: aaron.ballman, lebedev.ri, rnk, majnemer, zturner
Reviewed By: aaron.ballman
Subscribers: amccarth, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D73457

Fix failing bot

[mlir] NFC: use doxygen-style comments in AffineToStandard.cpp

Update tests for @llvm.memcpy.inline intrinsics

[llvm-readobj][test] - Remove --symbols --dyn-syms part from
Object/readobj-shared-object.test.
The intention of Object/readobj-shared-object.test was to check the 
general output for shared object.
I've added a case for testing dynamic objects to ELF/symbols.test. Also
we already test dynamic symbols printing in ELF/dyn-symbols.test + I've
added a case for `--dyn-syms` alias in D73164.
Hence we can remove this piece from Object/readobj-shared-object.test.
Differential revision: https://reviews.llvm.org/D73175

[VE] call isel with stack passing
Summary: Function calls and stack-passing of function arguments. Custom
lowering, isel patterns and tests.
Reviewed By: arsenm
Differential Revision: https://reviews.llvm.org/D73461

Add tanh lowering from Standard dialect to NVVM and ROCDL.
Summary: The tanh lowering from Standard dialect to NVVM and ROCDL was
not working. The conversion pattern are inserted in the lowering files. 
The test cases for the lowerings were added in the test files.
Reviewers: nicolasvasilache, ftynse, herhut
Reviewed By: ftynse, herhut
Subscribers: merge_guards_bot, ftynse, jholewinski, mehdi_amini,
rriddle, jpienaar, burmako, shauheen, antiagainst, csigg, arpith-jacob,
mgester, lucyrfox, herhut, liufengdb, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D73471

[mlir] fixed invalid LLVM intrinsics in LLVMOPs.td and
llvmir-intrinsics.mlir.
Summary: The intrinsic operation added multiple type annotations to the
llvm intrinsic operations, but only one is needed. The related tests in
llvmir-intrinsics.mlir checked the wrong number and are adjusted as
well.
Reviewers: nicolasvasilache, ftynse
Reviewed By: ftynse
Subscribers: merge_guards_bot, ftynse, mehdi_amini, rriddle, jpienaar,
burmako, shauheen, antiagainst, arpith-jacob, mgester, lucyrfox,
liufengdb, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D73470

[clangd] Improve ObjC property handling in SelectionTree.
Fixes https://github.com/clangd/clangd/issues/233
Reviewers: dgoldman
Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, jfb, kadircet,
usaxena95, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D72634

[lldb] Cut off unused suffix in CompletionRequest::GetRawLine
The GetRawLine currently returns the full command line used to create
the CompletionRequest. So for example for "foo b[tab] --arg" it would
return the whole string instead of "foo b". Usually completion code
makes the wrong assumption that the cursor is at the end of the line and
handing out the complete line will cause that people implement
completions that also make this assumption.
This patch makes GetRawLine() return only the string until the cursor
and hides the suffix (so that the cursor is always at the end of this
string) and adds another function GetRawLineWithUnusedSuffix that is
specifically the line with the suffix that isn't used by the
CompletionRequest for argument parsing etc.
There is only one user of this new function that actually needs the 
suffix and that is the expression command which needs the suffix to 
detect if it is in the raw or argument part of the command (by looking 
at the "--" separator).

[clangd] Support pseudo-obj expr, opaque values, and property references
in findExplicitReferences()
Reviewers: ilya-biryukov, dgoldman
Subscribers: MaskRay, jkorous, arphaman, jfb, kadircet, usaxena95,
cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D72508

[clangd][Hover] Handle uninstantiated templates
Summary: Fixes https://github.com/clangd/clangd/issues/263
Reviewers: hokein, sammccall
Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, usaxena95,
cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D73344

[lldb][NFC] Simplify Materializer/Dematerializer constructors

[AMDGPU] Fix vccz after v_readlane/v_readfirstlane to vcc_lo/hi
Summary: Up to gfx9, writes to vcc_lo and vcc_hi by instructions like 
v_readlane and v_readfirstlane do not update vccz to reflect the new 
value of vcc. Fix it by reusing part of the existing vccz bug handling 
code, which inserts an "s_mov_b64 vcc, vcc" instruction to restore vccz 
just before an instruction that needs the correct value.
Subscribers: arsenm, kzhuravl, jvesely, wdng, nhaehnle, yaxunl,
dstuttard, tpr, t-tye, hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69661

[mlir] NFC: use ValueRange in AffineToStandard conversion
ValueRange is a more flexible way of passing around ranges of Values 
that avoids Value vector materialization in affine expression expansion.

Add lowering for loop.parallel to cfg.
Summary: This also removes the explicit pattern for loop.terminator to
ensure that the terminator is only erased if the parent op is rewritten.
Reductions are not yet supported.
Reviewers: nicolasvasilache
Subscribers: mehdi_amini, rriddle, jpienaar, burmako, shauheen,
antiagainst, arpith-jacob, mgester, lucyrfox, aartbik, liufengdb,
llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D73348

[docs] Clarify llvm.used semantics with less awkward wording

[lldb/DWARF] Only match mangled name in full-name function lookup (with
accelerators)
Summary: In the spirit of https://reviews.llvm.org/D70846, we only
return functions with matching mangled name from
Apple/DebugNamesDWARFIndex::GetFunction if eFunctionNameTypeFull is
requested.
This speeds up lookup in the presence of large amount of class methods
of the same name (a typical examples would be constructors of templates
with many instantiations or overloaded operators).
Reviewers: labath
Reviewed By: labath
Subscribers: aprantl, arphaman, lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D73191

[clangd] use SCOPED_TRACE to better trace the testcase in test failure,
NFC
Reviewers: kadircet
Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, usaxena95,
cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D73463

[DebugInfo] Make most debug line prologue errors non-fatal to parsing
Many of the debug line prologue errors are not inherently fatal. In most 
cases, we can make reasonable assumptions and carry on. This patch does 
exactly that. In the case of length problems, the approach of "the 
claimed length is correct" is taken to be consistent with other 
instances such as the SectionParser, which ignores the read length.
Reviewed by: dblaikie
Differential Revision: https://reviews.llvm.org/D72158

Revert "[DebugInfo] Make most debug line prologue errors non-fatal to
parsing"
This reverts commit b94191fecdbadc18b342a27df1109754edcb8c4b.
The change broke both an LLD test and the LLDB build.

[LLD][ELF][ARM] clang-format function signature [NFC]
ARM::needsThunk had gone over 80 characters, run clang-format over it to 
prevent it wrapping.

[LLD][ELF][ARM] Do not insert interworking thunks for non STT_FUNC
symbols
ELF for the ARM architecture requires linkers to provide interworking
for symbols that are of type STT_FUNC. Interworking for other symbols
must be encoded directly in the object file. LLD was always providing
interworking, regardless of the symbol type, this breaks some programs
that have branches from Thumb state targeting STT_NOTYPE symbols that
have bit 0 clear, but they are in fact internal labels in a Thumb 
function. LLD treats these symbols as ARM and inserts a transition to
Arm.
This fixes the problem for in range branches, R_ARM_JUMP24, 
R_ARM_THM_JUMP24 and R_ARM_THM_JUMP19. This is expected to be the vast 
majority of problem cases as branching to an internal label close to the 
function.
There is at least one follow up patch required.
- R_ARM_CALL and R_ARM_THM_CALL may do interworking via BL/BLX
 substitution.
In theory range-extension thunks can be altered to not change state when 
the symbol type is not STT_FUNC. I will need to check with ld.bfd to see
if this is the case in practice.
Fixes (part of) https://github.com/ClangBuiltLinux/linux/issues/773
Differential Revision: https://reviews.llvm.org/D73474

[X86] Add combination for fma and fneg on X86 under strict FP.
Summary: X86 has instructions to calculate fma and fneg at the same
time. But we combine the fneg and fma only when fneg is the source
operand under strict FP.
Reviewers: craig.topper, andrew.w.kaylor, uweigand, RKSimon, LiuChen3
Subscribers: LuoYuanke, llvm-commits, cfe-commits, jdoerfert, hiraditya
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D72824

[mips][NFC] Remove unused instruction formats
`BranchBase` unused sice: rL170663
`FI` unsused since: rL170954
`FFI` unused since: rL190221
Differential revision: https://reviews.llvm.org/D73489

[FPEnv] Add pragma FP_CONTRACT support under strict FP.
Summary: Support pragma FP_CONTRACT under strict FP.
Reviewers: craig.topper, andrew.w.kaylor, uweigand, RKSimon, LiuChen3
Subscribers: hiraditya, jdoerfert, cfe-commits, llvm-commits, LuoYuanke
Tags: #clang, #llvm
Differential Revision: https://reviews.llvm.org/D72820

Revert "[lldb/DWARF] Only match mangled name in full-name function
lookup (with accelerators)"
This reverts commit 1b12766883006b8aa9d1ff744e57317647aa052a because of 
breaking the mac test suite.
I'm not certain this is the cause because of a concurrent build breakage 
which masked this problem, but the failure messages are related to 
symbol lookup, which makes this very likely.

[ADT] Implicitly convert between StringRef and std::string_view when we
have C++17
This makes the types almost seamlessly interchangeable in C++17 
codebases. Eventually we want to replace StringRef with the standard 
type, but that requires C++17 being the default and a huge refactoring 
job as StringRef has a lot more functionality.

[ASTMatchers] Add cxxNoexceptExpr AST matcher
Summary: Adds a cxxNoexceptExpr matcher that matches the [[
https://en.cppreference.com/w/cpp/language/noexcept |  noexcept operator
]].
Reviewers: aaron.ballman
Reviewed By: aaron.ballman
Subscribers: merge_guards_bot, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D73439

[NFC][RDA] typedef SmallPtrSetImpl<MachineInstr*>

Revert "PR44684: Look through parens and similar constructs when
determining"
This reverts commit af80b8ccc5772c14920d4554b7ca7e15f2fad1c4. It broke
clang-tidy/checkers/modernize-use-uncaught-exceptions.cpp in 
check-clang-tools on macOS and Windows, see e.g. 
http://lab.llvm.org:8011/builders/clang-x64-windows-msvc/builds/13976/steps/stage%201%20check/logs/stdio

Fix sphinx build bot failure. NFCI.

[lldb/PDB] Use the new line table constructor
The old method of adding line sequences one by one can easily go 
quadratic if the sequences are not perfectly sorted. The equivalent 
change in DWARF brought a considerable improvement in line table 
parsing. It is not clear if the same will be the case for PDB, but this 
does bring us a step closer towards removing the dangerous API.

[CodeMoverUtils] Improve IsControlFlowEquivalent.
Summary: Currently IsControlFlowEquivalent determine if two blocks are
control flow equivalent by checking if A dominates B and B post
dominates A. There exists blocks that are control flow equivalent even
if they don't satisfy the A dominates B and B post dominates A
condition. For example,
if (cond)
 A if (cond)
 B In the PR, we determine if two blocks are control flow equivalent by 
also checking if the two sets of conditions A and B depends on are 
equivalent. Reviewer: jdoerfert, Meinersbur, dmgreen, etiotto, bmahjour,
fhahn, hfinkel, kbarton Reviewed By: fhahn Subscribers: hiraditya,
llvm-commits Tag: LLVM Differential Revision:
https://reviews.llvm.org/D71578

[PowerPC][Future] Add pld and pstd to future CPU Add the prefixed
instructions pld and pstd to future CPU. These are load and store
instructions that require new operand types that are 34 bits. This patch
adds the two instructions as well as the operand types required.
Note that this patch also makes a minor change to tablegen to account
for the fact that some instructions are going to require shifts greater
than 31 bits for the new 34 bit instructions.
Differential Revision: https://reviews.llvm.org/D72574

Revert "[Clang] Warn about 'z' printf modifier in old MSVC."
This reverts commit fe0d1b6a8ac5048b8007e5e7cc2aeb4e3291bda0. Makes
Analysis/taint-generic.c fail on some Windows systems.

[clang-format] Handle quotes and escaped braces in C# interpolated
strings
Summary: This addresses issues raised in
https://bugs.llvm.org/show_bug.cgi?id=44454.
There are outstanding issues with multi-line verbatim strings in C# that
will be addressed in a follow-up PR.
Reviewers: krasimir, MyDeveloperDay
Reviewed By: krasimir, MyDeveloperDay
Subscribers: MyDeveloperDay
Tags: #clang-format
Differential Revision: https://reviews.llvm.org/D73492

Prevent building with MSVC 14.24
MSVC 14.24 miscompiles some of LLVM's code, which makes at least these
tests fail:
    LLVM :: MC/MachO/gen-dwarf-cpp.s
   LLVM :: MC/MachO/gen-dwarf-macro-cpp.s
   LLVM :: MC/MachO/gen-dwarf-producer.s
   LLVM :: MC/MachO/gen-dwarf.s
It seems better to diagnose that at build time. Since both the previous 
and the next version have a fix, this might be good enough and we might 
not need a real workaround. (We ran into this at 
https://crbug.com/1045948)
If you hit this, use either a newer or an older version of MSVC, or use
clang-cl as host compiler.
Differential Revision: https://reviews.llvm.org/D73550

[clangd] Make bin/llvm-lit run standalone clangd tests
Summary: Currently clangd lit tests can't be run in isolation because we
don't set some of the config parameters. This enables running
./bin/llvm-lit ../clang-tools-extra/clangd/test/
or any other test in that subdirectory.
Reviewers: sammccall
Subscribers: mgorny, ilya-biryukov, MaskRay, jkorous, arphaman,
usaxena95, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D73538

[mlir][spirv] Add GroupNonUniform arithmetic operations.
Add GroupNonUniform arithmetic operations: FAdd, FMul, IMul. Unify
parser, printer, verifier for GroupNonUniform arithmetic operations.
Differential Revision: https://reviews.llvm.org/D73491

[scudo][standalone] Secondary & general other improvements
Summary: This CL changes multiple things to improve performance (notably
on Android).We introduce a cache class for the Secondary that is taking 
care of this mechanism now.
The changes:
- change the Secondary "freelist" to an array. By keeping free secondary
 blocks linked together through their headers, we were keeping a page
 per block, which isn't great. Also we know touch less pages when
 walking the new "freelist".
- fix an issue with the freelist getting full: if the pattern is an ever
 increasing size malloc then free, the freelist would fill up and
 entries would not be used. So now we empty the list if we get to many
 "full" events;
- use the global release to os interval option for the secondary: it
 was too costly to release all the time, particularly for pattern that
 are malloc(X)/free(X)/malloc(X). Now the release will only occur
 after the selected interval, when going through the deallocate path;
- allow release of the `BatchClassId` class: it is releasable, we just
 have to make sure we don't mark the batches containing batches
 pointers as free.
- change the default release interval to 1s for Android to match the
 current Bionic allocator configuration. A patch is coming up to allow
 changing it through `mallopt`.
- lower the smallest class that can be released to `PageSize/64`.
Reviewers: cferris, pcc, eugenis, morehouse, hctim
Subscribers: phosek, #sanitizers, llvm-commits
Tags: #sanitizers, #llvm
Differential Revision: https://reviews.llvm.org/D73507

[clang][CodeComplete] Support for designated initializers
Reviewers: sammccall
Subscribers: cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D73271

[clang] Fix EOL whitespace. NFC

[ADT] Remove more llvm::make_unique
https://reviews.llvm.org/D73316

[gn build] Port a32f894f17b

Revert "[lldb/PDB] Use the new line table constructor"
This reverts commit bb73210ba9f16c1516f564235c86cbddccd1bd6d due to 
failures on the windows bot.

[HIP] Fix environment variable HIP_DEVICE_LIB_PATH
Currently device lib path set by environment variable
HIP_DEVICE_LIB_PATH does not work due to extra "-L" added to each entry.
This patch fixes that by allowing argument name to be empty in
addDirectoryList.
Differential Revision: https://reviews.llvm.org/D73299

[LV] Do not try to sink dead instructions.
Dead instructions do not need to be sunk. Currently we try and record 
the recipies for them, but there are no recipes emitted for them and 
there's nothing to sink. They can be removed from SinkAfter while 
marking them for recording.
Fixes PR44634.
Reviewers: rengolin, hsaito, fhahn, Ayal, gilr
Reviewed By: gilr
Differential Revision: https://reviews.llvm.org/D73423

Fix warning of `-Wcast-qual`. NFC.

[OPENMP50]Check for lastprivate conditional updates in atomic
constructs.
Added analysis in atomic constrcuts to support checks for updates of 
conditional lastprivate variables.

[LLD] Avoid exiting with a locked mutex NFC
In ErrorHandler::error(), rearrange code to avoid calling exitLld with 
the mutex locked. Acquire mutex lock when flushing the output streams in 
exitLld.
Differential Revision: https://reviews.llvm.org/D73281

Summary: This CL adds clang declarations of built-in functions for
AMDGPU MFMA intrinsics and instructions. OpenCL tests for new built-ins
are included.
Differential Revision: https://reviews.llvm.org/D72723

[LV] Hoist code to mark conditional assumes as dead to caller (NFC).
This is a follow-up suggested in D73423. It is sufficient to just add 
the conditional assumes to DeadInstructions once.

[llvm] Fix broken cases of 'CHECK[^:]*$' in tests

[lldb/Target] Add Assert StackFrame Recognizer
When a thread stops, this checks depending on the platform if the top
frame is an abort stack frame. If so, it looks for an assert stack frame
in the upper frames and set it as the most relavant frame when found.
To do so, the StackFrameRecognizer class holds a "Most Relevant Frame"
and a
"cooked" stop reason description. When the thread is about to stop, it
checks if the current frame is recognized, and if so, it fetches the
recognized frame's attributes and applies them.
rdar://58528686
Differential Revision: https://reviews.llvm.org/D73303
Signed-off-by: Med Ismail Bennani <medismail.bennani@gmail.com>

[Matrix] Mark expressions shared between multiple remarks.
This patch adds support for explicitly highlighting sub-expressions 
shared by multiple leaf nodes. For example consider the following code
  %shared.load = tail call <8 x double>
@llvm.matrix.columnwise.load.v8f64.p0f64(double* %arg1, i32 %stride, i32
2, i32 4), !dbg !10, !noalias !10
 %trans = tail call <8 x double> @llvm.matrix.transpose.v8f64(<8 x
double> %shared.load, i32 2, i32 4), !dbg !10
 tail call void @llvm.matrix.columnwise.store.v8f64.p0f64(<8 x double>
%trans, double* %arg3, i32 10, i32 4, i32 2), !dbg !10
 %load.2 = tail call <30 x double>
@llvm.matrix.columnwise.load.v30f64.p0f64(double* %arg3, i32 %stride,
i32 2, i32 15), !dbg !10, !noalias !10
 %mult = tail call <60 x double>
@llvm.matrix.multiply.v60f64.v8f64.v30f64(<8 x double> %trans, <30 x
double> %load.2, i32 4, i32 2, i32 15), !dbg !11
 tail call void @llvm.matrix.columnwise.store.v60f64.p0f64(<60 x double>
%mult, double* %arg2, i32 10, i32 4, i32 15), !dbg !11
We have two leaf nodes (the 2 stores) and the first store stores %trans 
which is also used by the matrix multiply %mult. We generate separate 
remarks for each leaf (stores). To denote that parts are shared, the 
shared expressions are marked as shared (), with a reference to the 
other remark that shares it. The operation summary also denotes the 
shared operations separately.
Reviewers: anemet, Gerolf, thegameg, hfinkel, andrew.w.kaylor, LuoYuanke
Reviewed By: anemet
Differential Revision: https://reviews.llvm.org/D72526

Add missing clang tests for 6d614a82a4230ea69e322f56dc18dcbd815ed37b
(AMDGPU MFMA built-ins)
Differential Revision: https://reviews.llvm.org/D72723

Revert "[lldb/Target] Add Assert StackFrame Recognizer"
This reverts commit 03a6b858fde5c644ec16b1fddd8e10aa9ef3f0ad.
The test doesn't pass on Debian.

[llvm-objcopy] Initial support for wasm in llvm-objcopy
Currently only supports simple copying, other operations to follow.
Reviewers: sbc100, alexshap, jhenderson
Differential Revision: https://reviews.llvm.org/D70930

[llvm-dwarfdump][Statistics] Distinguish functions/variables with same
name across different CUs
Different variables and functions might have the same name in different
CU. To calculate 'Availability' metric more accurate (i.e. to avoid
getting availability above 100%), we need to have some additional logic
to distinguish between them.
The patch introduces a DIE identifier that consists of a
function/variable name and declaration information: a filename and a
line number. This allows distinguishing different functions/variables
(different means declared in different files/lines) with the same name,
keeping duplicates counted as duplicates.
Reviewed by: aprantl, djtodoro
Differential Revision: https://reviews.llvm.org/D72797

[llvm-dwarfdump][Statistics] Ignore DW_TAG_subroutine_type in statistics
DW_TAG_subroutine_type is not really useful for statistics purposes, as
it never has location information. But it may contain
DW_TAG_formal_parameter children that generate number of parameters w/o
location and decrease
'availability' metric significantly.
Reviewed by: djtodoro
Differential Revision: https://reviews.llvm.org/D72983

[llvm-dwarfdump][Statistics] Ignore declarations of global variables
Reviewed by: djtodoro
Differential Revision: https://reviews.llvm.org/D73001

[WebAssembly] Add WebAssembly support to llvm-symbolizer
The only thing missing for basic llvm-symbolizer support is the ability
on lib/Object to get a wasm symbol's section ID, which allows sorting
and computation of the symbols' sizes.
Also, when the WasmAsmParser switches sections on new functions, also
add the section to the list of Dwarf sections if Dwarf is being
generated for assembly; this allows writing of simple tests.
Reviewers: sbc100, jhenderson, aardappel
Differential Revision: https://reviews.llvm.org/D73246

Fix a crash when casting _Complex and ignoring the results.
Performing a cast where the result is ignored caused Clang to crash when 
performing codegen for the conversion:
  _Complex int a;
 void fn1() { (_Complex double) a; }
This patch addresses the crash by not trying to emit the scalar
conversions, causing it to be a noop. Fixes PR44624.

[AArch64][GlobalISel] Don't bail out of the select(cmp(a, b)) -> csel
optimization with multiple users.
It can still be beneficial to do the optimization if the result of the
compare is used by *another* select.
Differential Revision: https://reviews.llvm.org/D73511

[clang-tidy] adjust scripts to subsubsections in Release Notes
Summary: I added subsubsections for typical Clang-tidy entries in
Release Notes, so now scripts are aware of this changes.
I don't have GitHub commit access, so please commit changes.
Reviewers: aaron.ballman, alexfh, hokein
Reviewed By: alexfh
Subscribers: njames93, xazax.hun, cfe-commits
Tags: #clang-tools-extra, #clang
Differential Revision: https://reviews.llvm.org/D72527

[CUDA] Assume the latest known CUDA version if we've found an unknown
one.
This makes clang somewhat forward-compatible with new CUDA releases 
without having to patch it for every minor release without adding any
new function.
If an unknown version is found, clang issues a warning (can be disabled 
with -Wno-cuda-unknown-version) and assumes that it has detected the
latest known version. CUDA releases are usually supersets of older ones
feature-wise, so it should be sufficient to keep released clang versions
working with minor CUDA updates without having to upgrade clang, too.
Differential Revision: https://reviews.llvm.org/D73231

Corrected clang amdgpu-features.cl test for
6d614a82a4230ea69e322f56dc18dcbd815ed37b (AMDGPU MFMA built-ins)
Differential Revision: https://reviews.llvm.org/D72723

[gn build] Port a928d127a52

[InstCombine] add tests for cmp with splat operand and splat constant;
NFC
See PR44588: https://bugs.llvm.org/show_bug.cgi?id=44588

[llvm-dwarfdump][Statistics] Count more than one conrete out-of-line
instances of a function
Here may be more than one out-of-line instance of the same function 
among different CUs. All of them should be accounted for to get an
accurate total number of variables/parameters.
Reviewed by: aprantl
Differential Revision: https://reviews.llvm.org/D73002

[llvm-dwarfdump][Statistics] Distinguish parameters with same name or
w/o a name
A few DW_TAG_formal_parameter's of the same function may have the same 
name (e.g. variadic (template) functions) or don't have a name at all
(if the parameter isn't used inside the function body), but we still 
need to be able to distinguish between them to get correct number of
'total vars' and 'availability' metric.
Reviewed by: aprantl
Differential Revision: https://reviews.llvm.org/D73003

[llvm-dwarfdump][Statistics] Make calculations of vars in global scope
more accurate
It isn't known how many times we've seen the same variable or member in 
the global scope (unlike in functions), but there still can be some
duplicates among different CUs. So, this patch proposes to count
variables in the global scope just as a sum of the number of vars,
constant members and artificial entities.
Reviewed by: aprantl
Differential Revision: https://reviews.llvm.org/D73004

[lldb/Bindings] Sort headers in headers.swig

Fix test hip-device-libs.hip

[MBFI] Move BranchFolding::MBFIWrapper to its own files. NFC.
Summary: To avoid header file circular dependency issues in passing
updated MBFI (in MBFIWrapper) to the interface of profile guided size
optimizations.
A prep step for (and split off of) D73381.
Reviewers: davidxl
Subscribers: mgorny, hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D73494

[gn build] Port 2c03c899d50

[scudo][standalone] Fix Android logging
Summary: Zygote & children's stderr is lost, so use Bionic's provided
allocation free syslog function for `outputRaw`. Get rid of the mutex as
it's not vital and could cause issues with `fork`.
Reviewers: cferris, pcc, eugenis, hctim, morehouse
Subscribers: #sanitizers, llvm-commits
Tags: #sanitizers, #llvm
Differential Revision: https://reviews.llvm.org/D73561

[ELF][PPC32] Support --emit-relocs link of R_PPC_PLTREL24
Similar to R_MIPS_GPREL16 and R_MIPS_GPREL32 (D45972).
If the addend of an R_PPC_PLTREL24 is >= 0x8000, it indicates that r30 
is relative to the input section .got2.
``` addis 30, 30, .got2+0x8000-.L1$pb@ha addi 30, 30,
.got2+0x8000-.L1$pb@l
... bl foo+0x8000@PLT
```
After linking, the relocation will be relative to the output section
.got2. To compensate for the shift `address(input section .got2) -
address(output section .got2) = ppc32Got2OutSecOff`, adjust by
`ppc32Got2OutSecOff`:
``` addis 30, 30, .got2+0x8000-.L1+ppc32Got2OutSecOff$pb@ha addi 30, 30,
.got2+0x8000-.L1+ppc32Got2OutSecOff$pb@ha$pb@l
... bl foo+0x8000+ppc32Got2OutSecOff@PLT
```
This rule applys to a relocatable link or a non-relocatable link with
--emit-relocs.
Reviewed By: Bdragon28
Differential Revision: https://reviews.llvm.org/D73532

llvm-symbolizer test: Add a bit of extra detail on how to
compile/reproduce this
The details are also in the .test file, but doesn't hurt to make it a 
bit clearer.

[mlir] Optimize OpResult use case for single result operations.
Summary: Operation represents all of the uses of each result with one
use list, so manipulating the use list of a specific result requires
filtering the main use list. This revision adds an optimization for the
case of single result operations to avoid this filtering.
Differential Revision: https://reviews.llvm.org/D73430

[mlir] Add primitive transform pattern to rewrite linalg.fill into
vector.broadcast form.
Summary: This diff adds a transformation patter to rewrite linalg.fill
as broadcasting a scaler into a vector. It uses the same preconditioning
as matmul (memory is contiguous).
Reviewers: nicolasvasilache
Subscribers: mehdi_amini, rriddle, jpienaar, burmako, shauheen,
antiagainst, arpith-jacob, mgester, lucyrfox, aartbik, liufengdb,
llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D73391

[Fuchsia] Remove aarch64-fuchsia target-specific -mcmodel=kernel
Under --target=aarch64-fuchsia, -mcmodel=kernel has the effect of
(the default) -mcmodel=small plus -mtp=el1 (which did not exist when 
this behavior was added). Fuchsia's kernel now uses -mtp=el1 directly
instead of -mcmodel=kernel, so remove this special support.
Patch By: mcgrathr
Differential Revision: https://reviews.llvm.org/D73409

[lsan] Support LeakSanitizer runtime on Fuchsia
Support LeakSanitizer runtime on Fuchsia.
Patch By: mcgrathr
Differential Revision: https://reviews.llvm.org/D72887

Don't fail step out if remote server doesn't implement qMemoryRegionInfo
Summary: The return address validation in D71372 will fail if the memory
permissions can't be determined. Many embedded stubs either don't
implement the qMemoryRegionInfo packet, or don't have memory permissions
at all.
Remove the return from the if clause that calls
GetLoadAddressPermissions, so this call failing doesn't cause the step
out to abort. Instead, assume that the memory permission check doesn't
apply to this type of target.
Reviewers: labath, jingham, clayborg, mossberg
Reviewed By: labath
Subscribers: lldb-commits
Tags: #lldb
Differential Revision: https://reviews.llvm.org/D72513

[CMake] Set ASM compiler for external projects
This is necessary on Windows, otherwise CMake fails. It's not 
conventional on Windows to use cl for assembly (you'd use ml or ml64 
instead), but CMake has a separate ASM_MASM mode for that, and clang-cl 
works fine for assembly so we'll use that on Windows for consistency.
Differential Revision: https://reviews.llvm.org/D73522

[InstCombine] refactor foldVectorCmp(); NFC
We can handle other patterns here as shown in PR44588.

Fix compiling with clang-cl inside a Visual Studio 2019 16.4 command
prompt.
This was introduced by 0d17410e919124b3a9194b9bdbe2ce39dd779941 and was
preventing from compiling with clang-cl on Windows. The problem was that
clang-cl detects the triple from the current env vars (was
x86_64-pc-windows-msvc19.24.28315 for me, as I happen to always run
inside a VS2019 cmd prompt).

[Instrumentation] Set hidden visibility for the bias variable
We have to avoid using a GOT relocation to access the bias variable, 
setting the hidden visibility achieves that.
Differential Revision: https://reviews.llvm.org/D73529

[NFCI][LoopUnrollAndJam] Minor changes.
Summary: 1. Add assertions. 2. Verify more analyses. These changes are
moved out of https://reviews.llvm.org/D73129 to simplify that review.
Reviewer: dmgreen, jdoerfert, Meinersbur, kbarton, bmahjour, etiotto 
Reviewed By: dmgreen Subscribers: fhahn, hiraditya, zzheng,
llvm-commits, prithayan, anhtuyen Tag: LLVM Differential Revision:
https://reviews.llvm.org/D73204

[LoopUnrollAndJamPass] Clean unnecessary includes. [NFCI]

[mlir][Linalg][doc] Add Design Document for the Linalg Dialect

[gn build] (manually) port 90a10f00ff8

[lldb] [test] Restrict x86-64-write register test to x86-64

[lldb/Reproducer] Include deserialized value in log statement
Extend the replay log statement to include the deserialized value.

Revert "[MS] Overhaul how clang passes overaligned args on x86_32"
It broke some Chromium tests, so let's revert until it can be fixed; see 
https://crbug.com/1046362
This reverts commit 2af74e27ed7d0832cbdde9cb969aaca7a42e99f9.

[GlobalMerge] Preserve symbol visibility when merging globals
Symbols created for merged external global variables have default 
visibility. This can break programs when compiling with -Oz
-fvisibility=hidden as symbols that should be hidden will be exported at 
link time.
Differential Revision: https://reviews.llvm.org/D73235

[llvm] Ensure InlineCost-related fields are initialized
Summary: Small fix - never hurts to have things initialized.
Reviewers: davidxl, eraman
Reviewed By: davidxl
Subscribers: haicheng, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D73420

[lldb] Remove ClangASTImporter from Target
Target is one of the classes responsible for vending ClangASTImporter. 
Target doesn't need to know anything about ClangASTImporter, so if we 
instead have ClangPersistentVariables vend it, we can preserve existing
behavior while improving layering and removing dependencies from
non-plugins to plugins.

[CodeGen] Attach no-builtin attributes to function definitions with no
Decl
When using -fno-builtin[-<name>], we don't attach the IR attributes to 
function definitions with no Decl, like the ones created through
`CreateGlobalInitOrDestructFunction`.
This results in projects using -fno-builtin or -ffreestanding to start 
seeing symbols like _memset_pattern16.
The fix changes the behavior to always add the attribute if LangOptions 
requests it.
Differential Revision: https://reviews.llvm.org/D73495

[AArch64][GlobalISel] Fold G_AND into G_BRCOND
When the G_BRCOND is fed by a eq or ne G_ICMP, it may be possible to
fold a G_AND into the branch by producing a tbnz/tbz instead.
This happens when
  1. We have a ne/eq G_ICMP feeding into the G_BRCOND
 2. The G_ICMP is a comparison against 0
 3. One of the operands of the G_AND is a power of 2 constant
This is very similar to the code in AArch64TargetLowering::LowerBR_CC.
Add opt-and-tbnz-tbz to test this.
Differential Revision: https://reviews.llvm.org/D73573

[AArch64][SVE] Add SVE2 mla indexed intrinsics.
Summary: Add SVE2 mla indexed intrinsics:
- smlalb, smalalt, umlalb, umlalt, smlslb, smlslt, umlslb, umlslt.
Reviewers: efriedma, sdesmalen, dancgr, cameron.mcinally, c-rhodes,
rengolin
Subscribers: tschuett, kristof.beyls, hiraditya, rkruppe, arphaman,
psnobl, llvm-commits, amehsan
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D73576

[lldb] Delete ValueObject::GetBaseClassPath
Summary: This method has exactly one call site, which is only actually
executed if `ValueObject::IsBaseClass` returns false. However, the first
thing that `ValueObject::GetBaseClassPath` does is check if
`ValueObject::IsBaseClass` is true. Because this can never be the case,
this method always returns false and is therefore effectively dead.
Differential Revision: https://reviews.llvm.org/D73517

Make llvm::StringRef to std::string conversions explicit.
This is how it should've been and brings it more in line with 
std::string_view. There should be no functional change here.
This is mostly mechanical from a custom clang-tidy check, with a lot of 
manual fixups. It uncovers a lot of minor inefficiencies.
This doesn't actually modify StringRef yet, I'll do that in a follow-up.