SuccessChanges

Summary

  1. jenkins/tasks: virtualenv: Use the Python3 interpreter (details)
Commit 92122d159f7294fc3534d16354385cd42837ff4c by Azharuddin Mohammed
jenkins/tasks: virtualenv: Use the Python3 interpreter
The file was modifiedtasks/utils/venv.sh (diff)

Summary

  1. Revert "[Clang] -Wunused-but-set-parameter and -Wunused-but-set-variable" (details)
  2. [NFC][scudo] Clang-format tests (details)
  3. Revert "X86: support Swift Async context" (details)
  4. [lldb] Document ctrl-f for completing show-autosuggestion (details)
  5. [lld][MinGW] Introduce aliases for -Bdynamic and -Bstatic (details)
  6. [clang CodeGen] Don't crash on large atomic function parameter. (details)
  7. [Driver][test] Don't assume integrated-as (details)
  8. [mlir][NFC] Remove stale `createLowerAffinePass` declaration (details)
  9. [WebAssembly] Nullify DBG_VALUE_LISTs in DebugValueManager (details)
  10. [test] Free triple in PassBuilderBindingsTest (details)
  11. Do actual DCE in LoopUnroll (try 3) (details)
  12. [AArch64][SVE] Implement extractelement of i1 vectors. (details)
Commit 3a0b6dc3e84bb91c0dbd721b1931e4a0ff396142 by aeubanks
Revert "[Clang] -Wunused-but-set-parameter and -Wunused-but-set-variable"

This reverts commit 14dfb3831c425c7f22540a2160424224008c257e.

More false positives, see D100581.
The file was modifiedclang/test/CodeGen/2007-10-30-Volatile.c
The file was modifiedclang/test/SemaCXX/sizeless-1.cpp
The file was modifiedclang/test/SemaCXX/goto.cpp
The file was removedclang/test/SemaCXX/warn-unused-but-set-parameters-cpp.cpp
The file was modifiedclang/include/clang/Basic/DiagnosticSemaKinds.td
The file was modifiedclang/test/CodeGen/X86/x86_32-xsave.c
The file was modifiedclang/test/Sema/vector-gcc-compat.c
The file was modifiedclang/test/CodeGen/builtins-riscv.c
The file was removedclang/test/Sema/warn-unused-but-set-variables.c
The file was modifiedclang/test/Sema/shift.c
The file was modifiedclang/test/FixIt/fixit.cpp
The file was modifiedclang/test/SemaCXX/shift.cpp
The file was modifiedclang/lib/Sema/SemaDecl.cpp
The file was modifiedclang/lib/Sema/SemaExprCXX.cpp
The file was modifiedclang/test/CodeGen/X86/x86_64-xsave.c
The file was modifiedclang/include/clang/Basic/DiagnosticGroups.td
The file was modifiedclang/include/clang/Sema/Sema.h
The file was modifiedclang/test/SemaObjC/foreach.m
The file was removedclang/test/Sema/warn-unused-but-set-parameters.c
The file was modifiedclang/lib/Sema/SemaExpr.cpp
The file was modifiedclang/test/CodeGen/builtins-arm.c
The file was modifiedclang/test/CXX/expr/expr.prim/expr.prim.lambda/p12.cpp
The file was modifiedclang/test/Sema/vector-gcc-compat.cpp
The file was modifiedclang/test/Misc/warning-wall.c
The file was removedclang/test/SemaCXX/warn-unused-but-set-variables-cpp.cpp
Commit 1eb78a64c4a5d3febd1799643e0d0aa7151d52bf by Vitaly Buka
[NFC][scudo] Clang-format tests
The file was modifiedcompiler-rt/test/scudo/mismatch.cpp
The file was modifiedcompiler-rt/test/scudo/double-free.cpp
The file was modifiedcompiler-rt/test/scudo/quarantine.c
The file was modifiedcompiler-rt/test/scudo/secondary.c
The file was modifiedcompiler-rt/test/scudo/threads.c
The file was modifiedcompiler-rt/test/scudo/valloc.c
The file was modifiedcompiler-rt/test/scudo/memalign.c
The file was modifiedcompiler-rt/test/scudo/dealloc-race.c
The file was modifiedcompiler-rt/test/scudo/tsd_destruction.c
The file was modifiedcompiler-rt/test/scudo/random_shuffle.cpp
The file was modifiedcompiler-rt/test/scudo/malloc.cpp
The file was modifiedcompiler-rt/test/scudo/realloc.cpp
The file was modifiedcompiler-rt/test/scudo/preinit.c
The file was modifiedcompiler-rt/test/scudo/rss.c
The file was modifiedcompiler-rt/test/scudo/sized-delete.cpp
The file was modifiedcompiler-rt/test/scudo/options.cpp
The file was modifiedcompiler-rt/test/scudo/aligned-new.cpp
The file was modifiedcompiler-rt/test/scudo/interface.cpp
The file was modifiedcompiler-rt/test/scudo/overflow.c
The file was modifiedcompiler-rt/test/scudo/stats.c
The file was modifiedcompiler-rt/test/scudo/alignment.c
Commit 6791a6b309b7aca847336f0175e97ff177687c7e by 31459023+hctim
Revert "X86: support Swift Async context"

This reverts commit 747e5cfb9f5d944b47fe014925b0d5dc2fda74d7.

Reason: New frame layout broke the sanitizer unwinder. Not clear why,
but seems like some of the changes aren't always guarded by Swyft
checks. See
https://reviews.llvm.org/rG747e5cfb9f5d944b47fe014925b0d5dc2fda74d7 for
more information.
The file was modifiedllvm/lib/Target/X86/X86MachineFunctionInfo.h
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.cpp
The file was modifiedllvm/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp
The file was removedllvm/test/CodeGen/X86/swift-async-reg.ll
The file was modifiedllvm/lib/Target/X86/X86FastISel.cpp
The file was modifiedllvm/lib/Target/X86/X86FrameLowering.cpp
The file was removedllvm/test/CodeGen/X86/swift-async.ll
Commit 02286d96dbb3862a7dc5f7a2481c1f763186b259 by davelee.com
[lldb] Document ctrl-f for completing show-autosuggestion

Document how to complete command line suggestions provided by `show-autosuggestion`.

Differential Revision: https://reviews.llvm.org/D102544
The file was modifiedlldb/source/Core/CoreProperties.td
Commit 65271ffe84165b935cefc9a7d468cb6f46a96a91 by markus.boeck02
[lld][MinGW] Introduce aliases for -Bdynamic and -Bstatic

Besides -Bdynamic and -Bstatic, ld documents additional aliases for both of these options. Instead of -Bstatic, one may write -dn, -non_shared or -static. Instead of -Bdynamic one may write -dy or -call_shared. Source: https://sourceware.org/binutils/docs-2.36/ld/Options.html

This patch adds those aliases to the MinGW driver of lld for the sake of ld compatibility.

Encountered this case while compiling a static Qt 6.1 distribution and got build failures as -static was passed directly to the linker, instead of through the compiler driver.

Differential Revision: https://reviews.llvm.org/D102637
The file was modifiedlld/test/MinGW/lib.test
The file was modifiedlld/MinGW/Options.td
Commit 698568b74c93ab6d9374adc8bdc6e60fbcf41ff1 by efriedma
[clang CodeGen] Don't crash on large atomic function parameter.

I wouldn't recommend writing code like the testcase; a function
parameter isn't atomic, so using an atomic type doesn't really make
sense.  But it's valid, so clang shouldn't crash on it.

The code was assuming hasAggregateEvaluationKind(Ty) implies Ty is a
RecordType, which isn't true.  Just use isRecordType() instead.

Differential Revision: https://reviews.llvm.org/D102015
The file was modifiedclang/test/CodeGenCXX/atomic.cpp
The file was modifiedclang/test/CodeGen/big-atomic-ops.c
The file was modifiedclang/lib/CodeGen/CGDecl.cpp
The file was modifiedclang/lib/CodeGen/CGCall.cpp
Commit 82b52812471ca4e7228714aab15110bb740669d0 by Jinsong Ji
[Driver][test] Don't assume integrated-as

The tests of fdebug-compilation-dir and -ffile-compilation-dir for `-x
assembler` are assuming integrated-as.
If the platform set the no-itegrated-as by default (eg: AIX for now), then this test will
fail.

Add the -integrated-as to aviod relying on the platform defaults.

Reviewed By: thakis

Differential Revision: https://reviews.llvm.org/D102647
The file was modifiedclang/test/Driver/clang_f_opts.c
Commit e2e1a78abcefb396ea1c08990f4cf20ae5068ef8 by riddleriver
[mlir][NFC] Remove stale `createLowerAffinePass` declaration

This pass isn't defined in the Transforms/ library anymore.
The file was modifiedmlir/include/mlir/Transforms/Passes.h
Commit 6e1c1dac4c72cc57f4cd2bc8554e8ac9f2f50b6e by aheejin
[WebAssembly] Nullify DBG_VALUE_LISTs in DebugValueManager

WebAssemblyDebugValueManager class currently does not handle
DBG_VALUE_LIST instructions correctly for two reasons, which are
explained in https://bugs.llvm.org/show_bug.cgi?id=50361.

This effectively nullifies DBG_VALUE_LISTs in
WebAssemblyDebugValueManager so that the info will appear as "optimized
out" in debuggers but still be at least correct in the meantime.

Reviewed By: dschuff, jmorse

Differential Revision: https://reviews.llvm.org/D102589
The file was modifiedllvm/lib/Target/WebAssembly/WebAssemblyDebugValueManager.cpp
The file was modifiedllvm/lib/Target/WebAssembly/WebAssemblyExplicitLocals.cpp
The file was addedllvm/test/CodeGen/WebAssembly/reg-stackify-dbg.mir
Commit ceb1ac9812cb0972cfa34cecc5e5704ae2a22063 by aeubanks
[test] Free triple in PassBuilderBindingsTest
The file was modifiedllvm/unittests/Passes/PassBuilderBindingsTest.cpp
Commit b6320eeb8622f05e4a5d4c7f5420523357490fca by listmail
Do actual DCE in LoopUnroll (try 3)

Recommitting after fixing a bug found post commit.  Amusingly, try 1 had been correct, and by reverting to incorporate last minute review feedback, I introduce the bug.  Oops.  :)

The problem was that recursively deleting an instruction can delete instructions beyond the current iterator (via a dead phi), thus invalidating iteration.  Test case added in LoopUnroll/dce.ll to cover this case.

LoopUnroll does a limited DCE pass after unrolling, but if you have a chain of dead instructions, it only deletes the last one. Improve the code to recursively delete all trivially dead instructions.

Differential Revision: https://reviews.llvm.org/D102511
The file was modifiedllvm/test/Transforms/LoopUnroll/AArch64/full-unroll-trip-count-upper-bound.ll
The file was modifiedllvm/test/Transforms/LoopUnroll/optsize-loop-size.ll
The file was modifiedllvm/test/Transforms/LoopUnroll/scevunroll.ll
The file was modifiedllvm/test/Transforms/LoopUnroll/unroll-header-exiting-with-phis.ll
The file was modifiedllvm/test/Transforms/LoopUnroll/pr45939-peel-count-and-complete-unroll.ll
The file was addedllvm/test/Transforms/LoopUnroll/dce.ll
The file was modifiedllvm/test/Transforms/LoopUnroll/nonlatchcondbr.ll
The file was modifiedllvm/test/Transforms/LoopUnrollAndJam/unroll-and-jam.ll
The file was modifiedllvm/test/Transforms/LoopUnroll/full-unroll-invariant.ll
The file was modifiedllvm/test/Transforms/LoopUnroll/unroll-unconditional-latch.ll
The file was modifiedllvm/lib/Transforms/Utils/LoopUnroll.cpp
Commit 3dd49ec1948b8be0b025e1558071506a58454722 by efriedma
[AArch64][SVE] Implement extractelement of i1 vectors.

The implementation just extends the vector to a larger element type, and
extracts from that.  Not fancy, but generates reasonable code.

There was discussion in the review of doing the promotion in
target-independent code, but I'm sticking with this to avoid making
LegalizeDAG infrastructure more complicated.

Differential Revision: https://reviews.llvm.org/D87651
The file was modifiedllvm/lib/Target/AArch64/AArch64ISelLowering.cpp
The file was modifiedllvm/test/CodeGen/AArch64/sve-extract-element.ll