FailedChanges

Summary

  1. [BareMetal] Ensure that sysroot always comes after library paths (details)
  2. Fix the module-enabled build by removing a redundant type definition. (details)
  3. [AArch64][GlobalISel] Legalize narrow type G_CTPOPs (details)
  4. [NewPM] Move analysis invalidation/clearing logging to instrumentation (details)
  5. NFC: Move TypeList implementation up the file (details)
  6. Make `hasTypeLoc` matcher support more node types. (details)
  7. [GlobalISel] Don't form zero/sign extending loads for atomics. (details)
Commit 167906c10932f5eda97b480ee084b17746c362e7 by phosek
[BareMetal] Ensure that sysroot always comes after library paths

This addresses an issue introduced in D91559. We would invoke the
compiler with -Lpath/to/lib --sysroot=path/to/sysroot where both
locations contain libraries with the same name, but we expect linker
to pick up the library in path/to/lib since that version is more
specialized. This was the case before D91559 where the sysroot path
would be ignored, but after that change linker would now pick up the
library from the sysroot which resulted in unexpected behavior.

The sysroot path should always come after any user provided library
paths, followed by compiler runtime paths. We want for libraries in user
provided library paths to always take precedence over sysroot libraries.
This matches the behavior of other toolchains used with other targets.

Differential Revision: https://reviews.llvm.org/D102049
The file was modifiedclang/lib/Driver/ToolChains/BareMetal.cpp
The file was modifiedclang/test/Driver/baremetal.cpp
Commit c6ddf669dcf379360e97a557e13617435d3c78cc by Adrian Prantl
Fix the module-enabled build by removing a redundant type definition.
The file was modifiedllvm/lib/Transforms/IPO/PassManagerBuilder.cpp
Commit 1312852040b3190a6cb7d7c1f61fe95a5e930d8d by Jessica Paquette
[AArch64][GlobalISel] Legalize narrow type G_CTPOPs

Using `clampScalar` here because we ought to mark s128 as custom eventually.

(Right now, it will just fall back.)

With this legalization, we get the same code as SDAG:
https://godbolt.org/z/TneoPKrKG

Differential Revision: https://reviews.llvm.org/D100908
The file was modifiedllvm/test/CodeGen/AArch64/GlobalISel/legalizer-info-validation.mir
The file was modifiedllvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp
The file was modifiedllvm/test/CodeGen/AArch64/GlobalISel/legalize-ctpop.mir
Commit 6f7131002b6a821fc9b245ec5179910f171e3358 by aeubanks
[NewPM] Move analysis invalidation/clearing logging to instrumentation

We're trying to move DebugLogging into instrumentation, rather than
being part of PassManagers/AnalysisManagers.

Reviewed By: ychen

Differential Revision: https://reviews.llvm.org/D102093
The file was modifiedllvm/unittests/IR/PassManagerTest.cpp
The file was modifiedllvm/unittests/IR/PassBuilderCallbacksTest.cpp
The file was modifiedllvm/include/llvm/IR/PassInstrumentation.h
The file was modifiedllvm/include/llvm/CodeGen/MachinePassManager.h
The file was modifiedllvm/unittests/Transforms/Scalar/LoopPassManagerTest.cpp
The file was modifiedllvm/unittests/CodeGen/PassManagerTest.cpp
The file was modifiedllvm/lib/LTO/LTOBackend.cpp
The file was modifiedllvm/lib/Passes/StandardInstrumentations.cpp
The file was modifiedllvm/unittests/Analysis/CGSCCPassManagerTest.cpp
The file was modifiedllvm/include/llvm/IR/PassManagerImpl.h
The file was modifiedllvm/tools/opt/NewPMDriver.cpp
The file was modifiedclang/lib/CodeGen/BackendUtil.cpp
The file was modifiedllvm/include/llvm/IR/PassManager.h
The file was modifiedllvm/test/Transforms/LoopUnroll/unroll-loop-invalidation.ll
Commit 0ad494838b8576de14144776490faa710fa2a099 by steveire
NFC: Move TypeList implementation up the file

This will make it possible for more code to use it.
The file was modifiedclang/include/clang/ASTMatchers/ASTMatchersInternal.h
Commit 1f65f42dd37ab6a950d3ec110e3efca0ace1b615 by steveire
Make `hasTypeLoc` matcher support more node types.

Differential Revision: https://reviews.llvm.org/D101572
The file was modifiedclang/unittests/ASTMatchers/ASTMatchersTraversalTest.cpp
The file was modifiedclang/include/clang/ASTMatchers/ASTMatchersInternal.h
The file was modifiedclang/docs/LibASTMatchersReference.html
The file was modifiedclang/include/clang/ASTMatchers/ASTMatchers.h
Commit 808bc11d9e1aa01edaf7ec4e56be3aee5ed42a83 by Amara Emerson
[GlobalISel] Don't form zero/sign extending loads for atomics.

For importing patterns, we only support matching G_LOAD, not G_ZEXTLOAD or
G_SEXTLOAD.

Differential Revision: https://reviews.llvm.org/D101932
The file was modifiedllvm/test/CodeGen/AArch64/GlobalISel/prelegalizercombiner-extending-loads.mir
The file was modifiedllvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp