SuccessChanges

Summary

  1. [SCEV] Remove unused getOrInsertCanonicalInductionVariable (NFC) (details)
  2. [Transforms] Use llvm::find_if (NFC) (details)
  3. [llvm] Drop unnecessary make_range (NFC) (details)
  4. [lld-macho] Remove unnecessary llvm:: namespace prefixes (details)
  5. [OpenMP] Remove copy constructor of `RTLInfoTy` (details)
  6. [gn build] Make an explicit `use_lld = true` on mac use lld.darwinnew (details)
  7. [SimplifyCFG] Keep !dgb metadata of moved instruction, if they match. (details)
  8. [mlir][vector] modified scatter/gather syntax, pass_thru mandatory (details)
  9. [NFC] Disallow unused prefixes in CodeGen/X86 tests. (details)
  10. [clang] Add llvm-strip to test dependencies (details)
Commit 9a7c03b80092d8efb57f7176d14b2bfa7743f245 by kazu
[SCEV] Remove unused getOrInsertCanonicalInductionVariable (NFC)

The last use was removed on Mar 22, 2012 in commit
f47d0af5515bee47dfd000363740fe347bb6cd18.
The file was modifiedllvm/include/llvm/Transforms/Utils/ScalarEvolutionExpander.h
The file was modifiedllvm/lib/Transforms/Utils/ScalarEvolutionExpander.cpp
Commit 4d92ab1669a4a74219c655e00c6de6ea03b16fe8 by kazu
[Transforms] Use llvm::find_if (NFC)
The file was modifiedllvm/lib/Transforms/Scalar/LICM.cpp
The file was modifiedllvm/lib/Transforms/Scalar/GVNHoist.cpp
The file was modifiedllvm/lib/Transforms/Scalar/MergeICmps.cpp
Commit 6a6e382161a9e37d8022de205bd1e1ac5762befd by kazu
[llvm] Drop unnecessary make_range (NFC)
The file was modifiedllvm/utils/TableGen/CodeGenSchedule.cpp
The file was modifiedllvm/utils/TableGen/InstrInfoEmitter.cpp
The file was modifiedllvm/lib/Analysis/LoopAccessAnalysis.cpp
The file was modifiedllvm/lib/CodeGen/MachineOutliner.cpp
The file was modifiedllvm/lib/Transforms/Scalar/LoopStrengthReduce.cpp
The file was modifiedllvm/tools/dsymutil/DebugMap.cpp
The file was modifiedllvm/lib/Transforms/Vectorize/VPlanPredicator.cpp
The file was modifiedllvm/lib/Analysis/StackSafetyAnalysis.cpp
Commit e98b441a09fac0a8419fee15afc725758390ca2a by jezng
[lld-macho] Remove unnecessary llvm:: namespace prefixes
The file was modifiedlld/MachO/LTO.cpp
The file was modifiedlld/MachO/InputFiles.cpp
The file was modifiedlld/MachO/Driver.cpp
The file was modifiedlld/MachO/OutputSegment.cpp
The file was modifiedlld/MachO/SymbolTable.cpp
Commit 175c336a1c5a7d4cf2e24ec5188c370cd6093ddb by tianshilei1992
[OpenMP] Remove copy constructor of `RTLInfoTy`

Multiple `RTLInfoTy` objects are stored in a list `AllRTLs`. Since
`RTLInfoTy` contains a `std::mutex`, it is by default not a copyable object.
In order to support `AllRTLs.push_back(...)` which is currently used, a customized
copy constructor is provided. Every time we need to add a new data member into
`RTLInfoTy`, we should keep in mind not forgetting to add corresponding assignment
in the copy constructor. In fact, the only use of the copy constructor is to push
the object into the list, we can of course write it in a way that first emplace
a new object back, and then use the reference to the last element. In this way we
don't need the copy constructor anymore. If the element is invalid, we just need
to pop it, and that's what this patch does.

Reviewed By: JonChesterfield

Differential Revision: https://reviews.llvm.org/D94361
The file was modifiedopenmp/libomptarget/src/rtl.cpp
The file was modifiedopenmp/libomptarget/src/rtl.h
Commit 87d4ea2433663b4aabe9c979a4dd47ef6adedc8b by thakis
[gn build] Make an explicit `use_lld = true` on mac use lld.darwinnew

use_lld defaults to true on non-mac if clang_base_path is set (i.e.
the host compiler is a locally-built clang). On mac, the lld Mach-O
port used to be unusable, but ld64.lld.darwinnew is close to usable.
When explicitly setting `use_lld = true` in a GN build on a mac host,
check-lld passes, two check-clang tests fail, and a handful check-llvm
tests fail (the latter all due to -flat_namespace not yet being implemented).
The file was modifiedllvm/utils/gn/build/BUILD.gn
Commit d98fc62ae68cd7e96bf6e85b7fbbee969cec9aa7 by flo
[SimplifyCFG] Keep !dgb metadata of moved instruction, if they match.

Currently SimplifyCFG drops the debug locations of 'bonus' instructions.
Such instructions are moved before the first branch. The reason for the
current behavior is that this could lead to surprising debug stepping,
if the block that's folded is dead.

In case the first branch and the instructions to be folded have the same
debug location, this shouldn't be an issue and we can keep the debug
location.

Reviewed By: vsk

Differential Revision: https://reviews.llvm.org/D93662
The file was modifiedllvm/lib/Transforms/Utils/SimplifyCFG.cpp
The file was modifiedllvm/test/Transforms/SimplifyCFG/fold-debug-location.ll
The file was modifiedllvm/docs/HowToUpdateDebugInfo.rst
Commit 6728af16cf987df3cf051f3a1f9c92ed2b8fbc2d by ajcbik
[mlir][vector] modified scatter/gather syntax, pass_thru mandatory

This change makes the scatter/gather syntax more consistent with
the syntax of all the other memory operations in the Vector dialect
(order of types, use of [] for index, etc.). This will make the MLIR
code easier to read. In addition, the pass_thru parameter of the
gather has been made mandatory (there is very little benefit in
using the implicit "undefined" values).

Reviewed By: nicolasvasilache

Differential Revision: https://reviews.llvm.org/D94352
The file was modifiedmlir/integration_test/Dialect/Vector/CPU/test-sparse-saxpy-jagged-matvec.mlir
The file was modifiedmlir/test/Conversion/VectorToLLVM/vector-to-llvm.mlir
The file was modifiedmlir/integration_test/Dialect/Vector/CPU/test-scatter.mlir
The file was modifiedmlir/integration_test/Dialect/Vector/CPU/test-sparse-dot-matvec.mlir
The file was modifiedmlir/test/Dialect/Vector/ops.mlir
The file was modifiedmlir/include/mlir/Dialect/Vector/VectorOps.td
The file was modifiedmlir/lib/Dialect/Vector/VectorOps.cpp
The file was modifiedmlir/test/Dialect/Vector/invalid.mlir
The file was modifiedmlir/test/Dialect/Vector/vector-mem-transforms.mlir
The file was modifiedmlir/integration_test/Dialect/Vector/CPU/test-gather.mlir
Commit 75c04327a5ec1f3b7014ebc410f3ef5833537863 by mtrofin
[NFC] Disallow unused prefixes in CodeGen/X86 tests.

Also fixed remaining tests that featured unused prefixes.

Differential Revision: https://reviews.llvm.org/D94330
The file was modifiedllvm/test/CodeGen/X86/fptosi-sat-scalar.ll
The file was modifiedllvm/test/CodeGen/X86/lit.local.cfg
The file was modifiedllvm/test/CodeGen/X86/vector-pack-128.ll
The file was modifiedllvm/test/CodeGen/X86/fptoui-sat-scalar.ll
Commit 4dbb3f57c6fe581ef525dbe49a401f4bab834eff by smeenai
[clang] Add llvm-strip to test dependencies

CodeGen/thinlto_embed_bitcode.ll relies on it.
The file was modifiedclang/test/CMakeLists.txt