FailedChanges

Summary

  1. [SCEV] Add more tests using info from loop guards for BTC. (details)
  2. [libc++] Initial support for pre-commit CI with Buildkite (details)
  3. [SCEV] Extract code to collect conditions to lambda (NFC). (details)
  4. Typo fix; NFC (details)
  5. [InstCombine] Add 'partial' bswap tests from PR39793 (details)
  6. Fix some of the more egregious 80-col and whitespace issues; NFC (details)
  7. [InstCombine] Add some extra bswap tests from PR39793 (details)
  8. [PowerPC][NFC] Merged two switch entries. (details)
  9. [SCEV] Swap operands if LHS is not unknown. (details)
  10. [AIX] Try to not use LLVM tools while building runtimes (details)
  11. [SVE] Lower fixed length VECREDUCE_[SMAX|SMIN] to Scalable (details)
  12. [SVE] Revert accidental change from 405e22fbe8719cff6c40eec15c2044f42527f116 (details)
  13. Add a verifier check that rejects non-distinct DISubprogram function (details)
  14. [NFC] Fix minor typos in comments and reuse concreteOp. (details)
  15. [NFC] Fix syntax of ranked memrefs in the MLIR Language Reference. (details)
  16. [libc++][ci] Don't require passing --token to phabricator-report (details)
Commit 1fa06162c1cf648a6d4fac837e02b709a205f4df by flo
[SCEV] Add more tests using info from loop guards for BTC.
The file was modifiedllvm/test/Analysis/ScalarEvolution/max-backedge-taken-count-guard-info.ll
Commit 9f21d341e83842c20f0cd09bb6b97617441ef55a by Louis Dionne
[libc++] Initial support for pre-commit CI with Buildkite

This commit adds basic files and scripts that are used for the Buildkite
pre-commit CI setup. This was tested to mostly work on a fork of llvm-project,
however some adjustments will have to be made as we complete the real
setup.
The file was addedlibcxx/utils/ci/phabricator-report
The file was addedlibcxx/utils/ci/buildkite-pipeline.yml
The file was addedlibcxx/utils/ci/secrets.env
The file was addedlibcxx/utils/ci/run-buildbot.sh
The file was addedlibcxx/utils/ci/Dockerfile
Commit df77ce7cad081bf55042cf098b61b118dcdfc7e9 by flo
[SCEV] Extract code to collect conditions to lambda (NFC).

This makes re-using the common functionality easier in follow-up
patches.
The file was modifiedllvm/lib/Analysis/ScalarEvolution.cpp
Commit 85cea77ecb7f2ca51198ec1ad1d28845e803ee32 by aaron
Typo fix; NFC
The file was modifiedclang/include/clang/Basic/AttrDocs.td
Commit 8d90d92f0dccfb9e1c150247128a28ed55fcb8ea by llvm-dev
[InstCombine] Add 'partial' bswap tests from PR39793

Tests for basic zext(bswap(trunc(x))) patterns shown on PR39793
The file was modifiedllvm/test/Transforms/InstCombine/bswap.ll
Commit a51d51a0d4d72ab423b11da4c5c877bc6e89a580 by aaron
Fix some of the more egregious 80-col and whitespace issues; NFC
The file was modifiedclang/include/clang/Basic/AttrDocs.td
Commit 132f29ce0611754e0ffb2b534c34b2ffe27b40a8 by llvm-dev
[InstCombine] Add some extra bswap tests from PR39793

Also test for cases where recognizeBSwapOrBitReverseIdiom checks for a truncated bswap pattern.
The file was modifiedllvm/test/Transforms/InstCombine/bswap.ll
Commit 8c53282d64b299a9543758988b1b480a88a93ae5 by stefanp
[PowerPC][NFC] Merged two switch entries.

Two switch entries did exactly the same thing. This patch merges them.
The file was modifiedlld/ELF/Arch/PPC64.cpp
Commit 8858340bd380b821bbad773c7b3a022a9bb5b2d9 by flo
[SCEV] Swap operands if LHS is not unknown.

Currently we only use information from guards for unknown expressions.
Swap LHS/RHS and predicate, if LHS is not unknown.
The file was modifiedllvm/lib/Analysis/ScalarEvolution.cpp
The file was modifiedllvm/test/Analysis/ScalarEvolution/max-backedge-taken-count-guard-info.ll
Commit e03dd978d015f6141d14afb3986f39365cabcdab by daltenty
[AIX] Try to not use LLVM tools while building runtimes

Since 64-bit XCOFF and the big AR format is not yet supported in some of these tools, this patch avoids additional setup of these tools. This patch is not intended to prevent picking up the LLVM tools if they happen to be available otherwise.

Reviewed By: hubert.reinterpretcast

Differential Revision: https://reviews.llvm.org/D85329
The file was modifiedllvm/runtimes/CMakeLists.txt
Commit e2ccf7f178ddaf202171a96cd6f41daa7a74a09c by mcinally
[SVE] Lower fixed length VECREDUCE_[SMAX|SMIN] to Scalable

This patch is pretty similar to the VECREDUCE_ADD patch, with some minor tweaks.

Results from the AArch64ISD::[SMAX|SMIN]V_PRED return element sized results. This requires an ANY_EXTEND for results < 32-bits, since Legalization promotes those results.

There is no NEON i64 vector support for SMAXV|SMINV, so use SVE for those.

Differential Revision: https://reviews.llvm.org/D88259
The file was modifiedllvm/test/CodeGen/AArch64/sve-fixed-length-int-reduce.ll
The file was modifiedllvm/lib/Target/AArch64/AArch64ISelLowering.cpp
Commit 9a4767411e89d35e55074e8783b909d0e8c6b2df by mcinally
[SVE] Revert accidental change from 405e22fbe8719cff6c40eec15c2044f42527f116

Accidentally commited two lines that were not intended. Remove those.
The file was modifiedllvm/lib/Target/AArch64/AArch64ISelLowering.cpp
Commit e17f52d623cc146b7d9bf5a2e02965043508b4c4 by Adrian Prantl
Add a verifier check that rejects non-distinct DISubprogram function
attachments. They would crash the backend, which expects all
DISubprograms that are not part of the type system to have a unit field.

Clang right before https://reviews.llvm.org/D79967 would generate this
kind of broken IR.

rdar://problem/69534688
The file was addedllvm/test/Verifier/unique-disubprogram.ll
The file was modifiedllvm/test/DebugInfo/Generic/2009-11-03-InsertExtractValue.ll
The file was modifiedllvm/lib/IR/Verifier.cpp
Commit 0b7f03b98d82dac7224e00377329577634d92c17 by jurahul
[NFC] Fix minor typos in comments and reuse concreteOp.

Differential Revision: https://reviews.llvm.org/D88242
The file was modifiedmlir/include/mlir/Dialect/Linalg/IR/LinalgTraits.h
The file was modifiedmlir/include/mlir/Dialect/Linalg/IR/LinalgStructuredOps.td
Commit 8046ab04a75bcd3a455faafa1364c67726748a78 by jurahul
[NFC] Fix syntax of ranked memrefs in the MLIR Language Reference.

- Eliminate incorrect |
- Eliminate memspace0 as the memory spaces currently are integer literals and memory
  space 0 is not explicitly printed.

Differential Revision: https://reviews.llvm.org/D88171
The file was modifiedmlir/docs/LangRef.md
Commit 2a11a197af7e72725fe461ba9917756b1b09661a by Louis Dionne
[libc++][ci] Don't require passing --token to phabricator-report

The CONDUIT_TOKEN is already taken from the environment. Also, disable
reporting back to Phabricator for now until we're ready to start spamming
the results back. This still needs a bit of testing.
The file was modifiedlibcxx/utils/ci/phabricator-report
The file was modifiedlibcxx/utils/ci/buildkite-pipeline.yml