SuccessChanges

Summary

  1. [zorg] Use boolean flag to determine if building a unified standalone runtimes (details)
Commit 1d2328ff69a8503911e85140b0786429204a2113 by Xiangling.Liao
[zorg] Use boolean flag to determine if building a unified standalone runtimes

For AIX libcxx build, we use ../llvm/runtimes as build source and with
-DLLVM_ENABLE_RUNTIMES="libcxx;libcxxabi;libunwind" option, since we are using
the new runtimes build laided out by the "A vision for building the runtimes"
RFC and D93408.

Differential Revision: https://reviews.llvm.org/D97326
The file was modifiedbuildbot/osuosl/master/config/builders.py (diff)
The file was modifiedzorg/buildbot/builders/LibcxxAndAbiBuilder.py (diff)

Summary

  1. [InstrProfiling] Place __llvm_prf_vnodes and __llvm_prf_names in llvm.used on ELF (details)
  2. Fix build failure due to dump() (details)
  3. [ARM] Update and add extra WLS testing. NFC (details)
  4. [mlir][lldb] Fix several gcc warnings in mlir and lldb (details)
  5. [clang-tidy] Added option to uniqueptr delete release check (details)
  6. [AArch64] Fix emitting an AdrpAddLdr LOH when there's a potential clobber of the (details)
  7. [ARM] Rename pass to MVETPAndVPTOptimisationsPass (details)
  8. [CUDA] Remove `noreturn` attribute from __assertfail(). (details)
  9. [AArch64][GlobalISel] NFC: Remove dead G_BUILD_VECTOR legalization rule (details)
  10. GlobalISel: Remove dead code (details)
  11. [clang-tidy] Added command line option `fix-notes` (details)
  12. Fix test cxx-call-kernel.cpp (details)
Commit 04c3040f417683e7c31b3ee3381a3263106f48c5 by i
[InstrProfiling] Place __llvm_prf_vnodes and __llvm_prf_names in llvm.used on ELF

`__llvm_prf_vnodes` and `__llvm_prf_names` are used by runtime but not
referenced via relocation in the translation unit.

With `-z start-stop-gc` (D96914 https://sourceware.org/bugzilla/show_bug.cgi?id=27451),
the linker no longer lets `__start_/__stop_` references retain them.

Place `__llvm_prf_vnodes` and `__llvm_prf_names` in `llvm.used` to make
them retained by the linker.

This patch changes most existing `UsedVars` cases to `CompilerUsedVars`
to reflect the ideal state - if the binary format properly supports
section based GC (dead stripping), `llvm.compiler.used` should be sufficient.

`__llvm_prf_vnodes` and `__llvm_prf_names` are switched to `UsedVars`
since we want them to be unconditionally retained by both compiler and linker.

Behaviors on other COFF/Mach-O are not affected.

Differential Revision: https://reviews.llvm.org/D97649
The file was modifiedllvm/test/Instrumentation/InstrProfiling/icall.ll (diff)
The file was addedcompiler-rt/test/profile/Linux/Inputs/instrprof-value-merge.c
The file was modifiedcompiler-rt/test/profile/Linux/instrprof-value-merge.c (diff)
The file was addedcompiler-rt/test/profile/Linux/instrprof-value-merge-lld.c
The file was modifiedllvm/include/llvm/Transforms/Instrumentation/InstrProfiling.h (diff)
The file was modifiedllvm/lib/Transforms/Instrumentation/InstrProfiling.cpp (diff)
Commit 53d30381f54fbeef3e9ca7f9b52d29d235365f19 by Yaxun.Liu
Fix build failure due to dump()

Change-Id: I86b534223d63bf8bb8f49af5a64b300efbeba77b
The file was modifiedclang/lib/CodeGen/CGExpr.cpp (diff)
Commit 54e2876132ff568d7f46cd6f2b0bf23ce95dbe2e by david.green
[ARM] Update and add extra WLS testing. NFC
The file was modifiedllvm/test/CodeGen/Thumb2/LowOverheadLoops/while-loops.ll (diff)
The file was modifiedllvm/test/Transforms/HardwareLoops/scalar-while.ll (diff)
The file was modifiedllvm/test/CodeGen/Thumb2/LowOverheadLoops/branch-targets.ll (diff)
Commit 801067f4c09bb8c3b625991c5adcb69b58eabaa1 by stilis
[mlir][lldb] Fix several gcc warnings in mlir and lldb

These warnings are raised when compiling with gcc due to either having too few or too many commas, or in the case of lldb, the possibility of a nullptr.

Reviewed By: mehdi_amini

Differential Revision: https://reviews.llvm.org/D97586
The file was modifiedlldb/source/Commands/CommandObjectTrace.cpp (diff)
The file was modifiedmlir/tools/mlir-linalg-ods-gen/mlir-linalg-ods-yaml-gen.cpp (diff)
The file was modifiedmlir/lib/Conversion/VectorToSPIRV/VectorToSPIRV.cpp (diff)
Commit 8bfc14193170c95a49fca3e66aa077203783a137 by n.james93
[clang-tidy] Added option to uniqueptr delete release check

Adds an option, `PreferResetCall`, currently defaulted to `false`, to the check.
When `true` the check will refactor by calling the `reset` member function.

Reviewed By: aaron.ballman

Differential Revision: https://reviews.llvm.org/D97630
The file was modifiedclang-tools-extra/clang-tidy/readability/UniqueptrDeleteReleaseCheck.h (diff)
The file was modifiedclang-tools-extra/docs/ReleaseNotes.rst (diff)
The file was modifiedclang-tools-extra/docs/clang-tidy/checks/readability-uniqueptr-delete-release.rst (diff)
The file was modifiedclang-tools-extra/clang-tidy/readability/UniqueptrDeleteReleaseCheck.cpp (diff)
The file was modifiedclang-tools-extra/test/clang-tidy/checkers/readability-uniqueptr-delete-release.cpp (diff)
Commit b783aa89795635cbe7b25b4143b562931fcec9f6 by Amara Emerson
[AArch64] Fix emitting an AdrpAddLdr LOH when there's a potential clobber of the
def of the adrp before the ldr.

Apparently this pass used to have liveness analysis but it was removed for
scompile time reasons. This workaround prevents the LOH from being emitted
unless the ADD and LDR are adjacent.

Fixes https://github.com/JuliaLang/julia/issues/39820

Differential Revision: https://reviews.llvm.org/D97571
The file was modifiedllvm/lib/Target/AArch64/AArch64CollectLOH.cpp (diff)
The file was addedllvm/test/CodeGen/AArch64/loh-adrp-add-ldr-clobber.mir
Commit e880f8b88a5cbb4338347b11e8a926e01a15bbb5 by david.green
[ARM] Rename pass to MVETPAndVPTOptimisationsPass

This pass has for a while performed Tail predication as well as VPT
block optimizations. Rename the pass to make that clear.
The file was modifiedllvm/utils/gn/secondary/llvm/lib/Target/ARM/BUILD.gn (diff)
The file was addedllvm/lib/Target/ARM/MVETPAndVPTOptimisationsPass.cpp
The file was modifiedllvm/lib/Target/ARM/ARM.h (diff)
The file was modifiedllvm/lib/Target/ARM/CMakeLists.txt (diff)
The file was modifiedllvm/lib/Target/ARM/ARMTargetMachine.cpp (diff)
The file was removedllvm/lib/Target/ARM/MVEVPTOptimisationsPass.cpp
Commit 32e0645276230bb5b736e378860df3b92b1f4ba8 by tra
[CUDA] Remove `noreturn` attribute from __assertfail().

`noreturn` complicates control flow and tends to trigger a known bug in ptxas if
the assert is used within loops in sufficiently complicated code.
https://bugs.llvm.org/show_bug.cgi?id=27738

Differential Revision: https://reviews.llvm.org/D97708
The file was modifiedclang/lib/Headers/__clang_cuda_runtime_wrapper.h (diff)
Commit 3e8223b16515443a254ff6ad3453a82602b4b9a0 by Jessica Paquette
[AArch64][GlobalISel] NFC: Remove dead G_BUILD_VECTOR legalization rule

Remove a rule which allows larger scalar types than the destination vector
element type.

This appears to be irrelevant now that we have G_BUILD_VECTOR_TRUNC. Plus,
making a G_BUILD_VECTOR which satisfies this introduces a verifier failure
anyway.

Differential Revision: https://reviews.llvm.org/D97727
The file was modifiedllvm/lib/Target/AArch64/GISel/AArch64LegalizerInfo.cpp (diff)
Commit 0131498402acbae4cfb445a5a98fcf93b3a0e676 by Matthew.Arsenault
GlobalISel: Remove dead code

Generic code should probably not introduce G_INSERT/G_EXTRACT. The
mirror unpackRegs should also be removed, but AMDGPU still has a use
remaining which needs to be fixed.
The file was modifiedllvm/lib/CodeGen/GlobalISel/CallLowering.cpp (diff)
The file was modifiedllvm/include/llvm/CodeGen/GlobalISel/CallLowering.h (diff)
Commit abbe9e227ed31e5dde9bb7567bb9f0dd047689c6 by n.james93
[clang-tidy] Added command line option `fix-notes`

Added an option to control whether to apply the fixes found in notes attached to clang tidy errors or not.
Diagnostics may contain multiple notes each offering different ways to fix the issue, for that reason the default behaviour should be to not look at fixes found in notes.
Instead offer up all the available fix-its in the output but don't try to apply the first one unless `-fix-notes` is supplied.

Reviewed By: aaron.ballman

Differential Revision: https://reviews.llvm.org/D84924
The file was modifiedclang-tools-extra/docs/clang-tidy/index.rst (diff)
The file was modifiedclang-tools-extra/test/clang-tidy/checkers/misc-unused-using-decls.cpp (diff)
The file was modifiedclang-tools-extra/test/clang-tidy/infrastructure/alternative-fixes.cpp (diff)
The file was modifiedclang-tools-extra/clang-tidy/tool/ClangTidyMain.cpp (diff)
The file was modifiedclang-tools-extra/test/clang-tidy/checkers/misc-unused-using-decls-cxx17.cpp (diff)
The file was modifiedclang-tools-extra/test/clang-tidy/checkers/misc-definitions-in-headers.hpp (diff)
The file was modifiedclang-tools-extra/test/clang-tidy/infrastructure/clean-up-code.cpp (diff)
The file was modifiedclang-tools-extra/test/clang-tidy/checkers/readability-inconsistent-declaration-parameter-name.cpp (diff)
The file was modifiedclang-tools-extra/docs/ReleaseNotes.rst (diff)
The file was modifiedclang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.h (diff)
The file was modifiedclang-tools-extra/clang-tidy/ClangTidy.h (diff)
The file was modifiedclang-tools-extra/clang-tidy/ClangTidy.cpp (diff)
The file was modifiedclang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp (diff)
Commit 9ecbb34e1d2dc77e768638b6ecd94d63e451915d by Yaxun.Liu
Fix test cxx-call-kernel.cpp

Only test it with x86 since other target may have an ABI
making it difficult to test.

Change-Id: I85423c8bbbbbb8f24cb3ea4cb64a408069b4d61c
The file was modifiedclang/test/CodeGenCUDA/cxx-call-kernel.cpp (diff)