SuccessChanges

Summary

  1. Revert "Support compatibility with buildbot 2.10." (details)
Commit 93900f5122cc7087aa934424b1aa6339952a20c0 by gkistanova
Revert "Support compatibility with buildbot 2.10."

Temporarily revert commit 3f44862c69dd1766b51ea408d2cba6ec5c76c30a, as we are still with 2.8.4 in the production.

Will be re-applied after migration to 2.10 or later.
The file was modifiedzorg/buildbot/commands/LitTestCommand.py (diff)

Summary

  1. [Utils] Run non-filecheck runlines in-order in update_cc_test_checks (details)
  2. [libcxx] Update docs regarding the need for bash/posix tools for tests on Windows. NFC. (details)
  3. [LLD] [COFF] Actually include the exported comdat symbols (details)
  4. [llvm-readobj] [ARMWinEH] Try to resolve label symbols into regular ones (details)
  5. Mark Basic/TargetCXXABI.def as textual in the module map. (details)
  6. [Coroutines] Do not add alloca to the frame if the size is 0 (details)
  7. Add cal entry for MemorySSA syncs. (details)
  8. [WebAssembly] Mark abs of v2i64 as legal (details)
  9. [MC] Remove unneeded "in '.xxx' directive" from diagnostics (details)
  10. X86/GlobalISel: Rely on default assignValueToReg (details)
  11. GlobalISel: Fix missing newline in debug printing (details)
  12. [MC] Don't capitalize a floating point diagnostic (details)
  13. [libc++] Fix QoI bug with construction of std::tuple involving std::any (details)
  14. [AMDGPU] Disable the scalar IR, SDWA and load store vectorizer passes at -O1 (details)
  15. [libc++] Make sure std::to_address doesn't depend on P::element_type. (details)
  16. Revert "[InstSimplify] Added tests for PR50173, NFC" (details)
  17. [MC] Add MCAsmParser::parseComma to improve diagnostics (details)
Commit 92f2c39f915adc8f71be02d2604df4916069c9d4 by georgakoudis1
[Utils] Run non-filecheck runlines in-order in update_cc_test_checks

The script update_cc_test_checks runs all non-filechecked runlines before the filechecked ones. This creates problems since outputs of those non-filechecked runlines may conflict and that will fail the execution of update_cc_test_checks. This patch executes non-filechecked in the order specified in the test file to avoid this issue.

Reviewed By: jdoerfert

Differential Revision: https://reviews.llvm.org/D101683
The file was modifiedclang/test/utils/update_cc_test_checks/Inputs/exec-all-runlines.c
The file was modifiedclang/test/utils/update_cc_test_checks/Inputs/exec-all-runlines.c.expected
The file was modifiedllvm/utils/update_cc_test_checks.py
Commit e87fb6d3875569bbc2e729732433e4865cdd1eff by martin
[libcxx] Update docs regarding the need for bash/posix tools for tests on Windows. NFC.

After 39bbfb77264a4a7a216921c2b70a30ba0f27eb56, bash is no longer
a hard requirement.

Differential Revision: https://reviews.llvm.org/D101779
The file was modifiedlibcxx/docs/BuildingLibcxx.rst
Commit 82de4e075339f5ad8d68cfe31eb45b771d4750ae by martin
[LLD] [COFF] Actually include the exported comdat symbols

This is a followup to 2b01a417d7ccb001ccc1185ef5fdc967c9fac8d7;
previously the RVAs of the exported symbols from comdats were left
zero.

Thanks to Kleis Auke Wolthuizen for the fix suggestion and pointing
out the omission.

Differential Revision: https://reviews.llvm.org/D101615
The file was modifiedlld/test/COFF/export-all.s
The file was modifiedlld/COFF/Driver.cpp
Commit 70c4930637fa9a43eabcbde66e5a74262da1e46c by martin
[llvm-readobj] [ARMWinEH] Try to resolve label symbols into regular ones

Unwind info generated by MSVC tends to have relocations pointing at
static "label" symbols like "$LN4" instead of regular ones based on
the actual function's name. Try to resolve such symbols to a non-label
symbol if possible (ideally to an external symbol), to improve
the readability.

Differential Revision: https://reviews.llvm.org/D101567
The file was modifiedllvm/tools/llvm-readobj/ARMWinEHPrinter.cpp
The file was modifiedllvm/tools/llvm-readobj/ARMWinEHPrinter.h
The file was addedllvm/test/tools/llvm-readobj/COFF/arm64-unwind-preferred-symbol.yaml
Commit 6c3a10760da3707186d5204bd9283e99344fa90f by Adrian Prantl
Mark Basic/TargetCXXABI.def as textual in the module map.
The file was modifiedclang/include/clang/module.modulemap
Commit def86413d4c7bd088c3dbf445480a3eb2074da66 by lxfind
[Coroutines] Do not add alloca to the frame if the size is 0

This patch is to address https://bugs.llvm.org/show_bug.cgi?id=49916.
When the size of an alloca is 0, it will trigger an assertion in OptimizedStructLayout when being added to the frame.
Fix it by not adding it at all. We return index 0 (beginning of the frame) for all 0-sized allocas.

Differential Revision: https://reviews.llvm.org/D101841
The file was modifiedllvm/lib/Transforms/Coroutines/CoroFrame.cpp
The file was addedllvm/test/Transforms/Coroutines/coro-zero-alloca.ll
Commit b14c8f5f6eb96e70adb553cc8a4257524881e029 by asbirlea
Add cal entry for MemorySSA syncs.
The file was modifiedllvm/docs/GettingInvolved.rst
Commit 14ca2e5e22e7806c77ca3e5b126e888c9b1c4041 by tlively
[WebAssembly] Mark abs of v2i64 as legal

We previously had an ISel pattern for i64x2.abs, but because the ISDNode was not
marked legal for v2i64, the instruction was not being selected.

Differential Revision: https://reviews.llvm.org/D101803
The file was modifiedllvm/test/CodeGen/WebAssembly/simd-arith.ll
The file was modifiedllvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp
Commit 3d473ae72ea3f1bff6db6a3ebc226f3032cf530d by i
[MC] Remove unneeded "in '.xxx' directive" from diagnostics

The directive name is not useful because the next line replicates the error line
which includes the directive.
The file was modifiedllvm/test/MC/ELF/cfi.s
The file was modifiedllvm/lib/MC/MCParser/AsmParser.cpp
The file was modifiedllvm/test/MC/AsmParser/AArch64/directive-parse-err.s
The file was modifiedllvm/test/MC/Mips/asciiz-directive-bad.s
The file was modifiedllvm/test/MC/AsmParser/exprs-invalid.s
The file was modifiedllvm/test/MC/RISCV/data-directives-invalid.s
The file was modifiedllvm/test/MC/ELF/lto-discard.s
The file was modifiedllvm/test/MC/AsmParser/equ.s
The file was modifiedllvm/test/MC/Hexagon/data-directives-invalid.s
The file was modifiedllvm/test/MC/COFF/cv-fpo-errors.s
The file was modifiedllvm/test/MC/AsmParser/floating-literals.s
Commit 6dd883477249dae1c57e07f6c4375eb90c2df297 by Matthew.Arsenault
X86/GlobalISel: Rely on default assignValueToReg

The resulting output is semantically closer to what the DAG emits and
is more compatible with the existing CCAssignFns.

The returns of f32 in f80 are clearly broken, but they were broken
before when using G_ANYEXT to go from f32 to f80.
The file was modifiedllvm/test/CodeGen/X86/GlobalISel/x86_64-irtranslator-struct-return.ll
The file was modifiedllvm/test/CodeGen/X86/GlobalISel/irtranslator-callingconv.ll
The file was modifiedllvm/test/CodeGen/X86/GlobalISel/x86_64-irtranslator.ll
The file was modifiedllvm/lib/Target/X86/X86CallLowering.cpp
Commit ccfe0175109af904942672dcbb0dd32963f71338 by Matthew.Arsenault
GlobalISel: Fix missing newline in debug printing
The file was modifiedllvm/lib/Target/AArch64/GISel/AArch64CallLowering.cpp
Commit 7b1e1fccb02af234cca44d84bfb045faef9e21c8 by i
[MC] Don't capitalize a floating point diagnostic
The file was modifiedllvm/test/MC/AsmParser/floating-literals.s
The file was modifiedllvm/lib/MC/MCParser/AsmLexer.cpp
Commit 17f2d1cb9b93d336d4187cd14307bef1ab535808 by Louis Dionne
[libc++] Fix QoI bug with construction of std::tuple involving std::any

In std::tuple, we should try to avoid calling std::is_copy_constructible
whenever we can to avoid surprising interactions with (I believe) compiler
builtins. This bug was reported in https://reviews.llvm.org/D96523#2730953.

The issue was that when tuple<_Up...> was the same as tuple<_Tp...>, we
would short-circuit the _Or (because sizeof...(_Tp) != 1) and go evaluate
the following `is_constructible<_Tp, const _Up&>...`. That shouldn't
actually be a problem, but see the analysis in https://reviews.llvm.org/D101770#2736470
for why it is with Clang and GCC.

Instead, after this patch, we check whether the constructed-from tuple
is the same as the current tuple regardless of the number of elements,
since we should always prefer the normal copy constructor in that case
anyway.

Differential Revision: https://reviews.llvm.org/D101770
The file was modifiedlibcxx/include/tuple
The file was addedlibcxx/test/std/utilities/tuple/tuple.tuple/tuple.cnstr/cnstr_with_any.compile.pass.cpp
Commit 6a17609157196878b9cd9aa9ce71bde247ca14db by baptiste.saleil
[AMDGPU] Disable the scalar IR, SDWA and load store vectorizer passes at -O1

This patch disables some of the passes at -O1. These passes have a significant
impact on compilation time, so we only want them to be enabled starting from -O2.

Differential Revision: https://reviews.llvm.org/D101414
The file was modifiedllvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
The file was addedllvm/test/CodeGen/AMDGPU/llc-pipeline.ll
Commit da456167f56a604810aaeb89d21d3c7047945566 by Louis Dionne
[libc++] Make sure std::to_address doesn't depend on P::element_type.

Differential Revision: https://reviews.llvm.org/D101638
The file was addedlibcxx/test/std/utilities/memory/pointer.conversion/to_address_std_iterators.pass.cpp
The file was addedlibcxx/test/libcxx/utilities/memory/pointer.conversion/to_address.pass.cpp
The file was modifiedlibcxx/include/iterator
The file was modifiedlibcxx/test/std/utilities/memory/pointer.conversion/to_address.pass.cpp
The file was modifiedlibcxx/include/__memory/pointer_traits.h
The file was addedlibcxx/test/libcxx/utilities/memory/pointer.conversion/to_address_std_iterators.pass.cpp
Commit 62fcda93788a93726f7bc7cb5c0699595e009ceb by Dávid Bolvanský
Revert "[InstSimplify] Added tests for PR50173, NFC"

This reverts commit 4e7a4c73dab6605f4fcc7bf09c2ee85e7925f6d7. Not needed, pattern is handled by instcombine already.
The file was modifiedllvm/test/Transforms/InstSimplify/call.ll
Commit 7cac6a9d7a1d637bf6a0701ec56c69d2b58fda69 by i
[MC] Add MCAsmParser::parseComma to improve diagnostics

llvm-mc will error "expected comma" instead of "unexpected token".
The file was modifiedllvm/test/MC/AsmParser/directive_dcb.s
The file was modifiedllvm/test/MC/ELF/cfi.s
The file was modifiedllvm/lib/MC/MCParser/AsmParser.cpp
The file was modifiedllvm/include/llvm/MC/MCParser/MCAsmParser.h

Summary

  1. Revert "Support compatibility with buildbot 2.10." (details)
Commit 93900f5122cc7087aa934424b1aa6339952a20c0 by gkistanova
Revert "Support compatibility with buildbot 2.10."

Temporarily revert commit 3f44862c69dd1766b51ea408d2cba6ec5c76c30a, as we are still with 2.8.4 in the production.

Will be re-applied after migration to 2.10 or later.
The file was modifiedzorg/buildbot/commands/LitTestCommand.py