FailedChanges

Summary

  1. [InstCombine] Add tests for D84948; NFC (details)
  2. [InstCombine] Fold freeze(undef) into a proper constant (details)
  3. [flang] Add parser support for OpenMP allocate clause (details)
  4. [mlir] Lower DimOp to LLVM for unranked memrefs. (details)
  5. [dsymutil] Disable dsymutil/X86/reproducer.test on windows. (details)
  6. [analyzer][tests] Understand when diagnostics change between builds (details)
  7. [MLIR] Change GpuLaunchFuncToGpuRuntimeCallsPass to wrap a RewritePattern with the same functionality. (details)
  8. [SVE] Implement lowering for fixed length vector multiplication. (details)
  9. [mlir][Linalg] Introduce canonicalization to remove dead LinalgOps (details)
  10. [SVE] Lower scalable vector mul operations. (details)
  11. [X86][SSE] Expose all memory offsets in expand load tests (details)
  12. PDBExtras.h - remove unnecessary raw_ostream forward declaration. NFCI. (details)
  13. [X86] getX86MaskVec - replace mask limit from NumElts < 8 with NumElts <= 4 (details)
  14. Fix include sorting order. NFC (details)
  15. Revert "[ELF] Allow sections after a non-SHF_ALLOC section to be covered by PT_LOAD" (details)
Commit 54a1097b8373ea609babb3cac5ba5708c9b26150 by aqjune
[InstCombine] Add tests for D84948; NFC
The file was modifiedllvm/test/Transforms/InstCombine/select.ll
The file was modifiedllvm/test/Transforms/InstCombine/freeze.ll
Commit c771087161f4948a4d56d64e433f6bbc9dbfad90 by aqjune
[InstCombine] Fold freeze(undef) into a proper constant

This is a simple patch that folds freeze(undef) into a proper constant after inspecting its uses.

Reviewed By: nikic

Differential Revision: https://reviews.llvm.org/D84948
The file was modifiedllvm/test/Transforms/InstCombine/select.ll
The file was modifiedllvm/test/Transforms/InstCombine/freeze-phi.ll
The file was modifiedllvm/test/Transforms/InstCombine/freeze.ll
The file was modifiedllvm/lib/Transforms/InstCombine/InstructionCombining.cpp
Commit e355f85bdf9e36a0f17428b281b7879051ed2867 by andrzej.warzynski
[flang] Add parser support for OpenMP allocate clause

Differential Revision: https://reviews.llvm.org/D85212
The file was modifiedflang/include/flang/Parser/dump-parse-tree.h
The file was modifiedflang/lib/Parser/unparse.cpp
The file was modifiedflang/lib/Parser/openmp-parsers.cpp
The file was modifiedflang/include/flang/Parser/parse-tree.h
The file was modifiedflang/test/Semantics/omp-clause-validity01.f90
Commit 3effc35015ce0163579943ba1c86c35d0a3ba752 by pifon
[mlir] Lower DimOp to LLVM for unranked memrefs.

Differential Revision: https://reviews.llvm.org/D85361
The file was modifiedmlir/lib/Dialect/StandardOps/IR/Ops.cpp
The file was modifiedmlir/lib/Conversion/StandardToLLVM/StandardToLLVM.cpp
The file was modifiedmlir/test/Conversion/StandardToLLVM/convert-to-llvmir.mlir
The file was modifiedmlir/test/mlir-cpu-runner/unranked_memref.mlir
Commit c6eb76093c3b749ea903cde210ad125f6b94853a by a.v.lapshin
[dsymutil] Disable dsymutil/X86/reproducer.test on windows.

The dsymutil/X86/reproducer.test test could create paths
longer than MAX_PATH:

C:\ps4-buildslave2\llvm-clang-x86_64-expensive-checks-win
\build\test\tools\dsymutil\X86\Output\reproducer.test.tmp.repro
\ps4-buildslave2\llvm-clang-x86_64-expensive-checks-win\build
\test\tools\dsymutil\X86\Output\reproducer.test.tmp\Inputs\
basic1.macho.x86_64.o

Disable this test on windows.

Reviewed By: JDevlieghere

Differential Revision: https://reviews.llvm.org/D85294
The file was modifiedllvm/test/tools/dsymutil/X86/reproducer.test
Commit 6ddef92474583ef3c183da9bdc8c8e81ec578fd8 by vsavchenko
[analyzer][tests] Understand when diagnostics change between builds

Before the patch `SATest compare`, produced quite obscure results
when something about the diagnostic have changed (i.e. its description
or the name of the corresponding checker) because it was simply two
lists of warnings, ADDED and REMOVED.  It was up to the developer
to match those warnings, understand that they are essentially the
same, and figure out what caused the difference.

This patch introduces another category of results: MODIFIED.
It tries to match new warnings against the old ones and prints out
clues on what is different between two builds.

Differential Revision: https://reviews.llvm.org/D85311
The file was modifiedclang/utils/analyzer/CmpRuns.py
Commit 45676a8936124e734177f1360e7af05a7ada7d99 by csigg
[MLIR] Change GpuLaunchFuncToGpuRuntimeCallsPass to wrap a RewritePattern with the same functionality.

The RewritePattern will become one of several, and will be part of the LLVM conversion pass (instead of a separate pass following LLVM conversion).

Reviewed By: herhut

Differential Revision: https://reviews.llvm.org/D84946
The file was modifiedmlir/lib/Conversion/GPUCommon/CMakeLists.txt
The file was modifiedmlir/include/mlir/Conversion/GPUCommon/GPUCommonPass.h
The file was modifiedmlir/lib/Conversion/GPUCommon/ConvertLaunchFuncToRuntimeCalls.cpp
The file was modifiedmlir/tools/mlir-cuda-runner/mlir-cuda-runner.cpp
The file was modifiedmlir/tools/mlir-rocm-runner/mlir-rocm-runner.cpp
Commit 3ed59b775d7ce54ef8e881b31ce8917b84470dfb by paul.walker
[SVE] Implement lowering for fixed length vector multiplication.

NOTE: Also uses SVE code generation for NEON size vectors, instead
of expanding i64 based vector multiplications.

Differential Revision: https://reviews.llvm.org/D85327
The file was modifiedllvm/test/CodeGen/AArch64/sve-fixed-length-int-arith.ll
The file was modifiedllvm/lib/Target/AArch64/AArch64ISelLowering.h
The file was modifiedllvm/lib/Target/AArch64/AArch64SVEInstrInfo.td
The file was modifiedllvm/lib/Target/AArch64/SVEInstrFormats.td
The file was modifiedllvm/lib/Target/AArch64/AArch64ISelLowering.cpp
Commit 54fafd17a728f3dd33b3cf999b6dfd3cd1d49f12 by ntv
[mlir][Linalg] Introduce canonicalization to remove dead LinalgOps

When any of the memrefs in a structured linalg op has a zero dimension, it becomes dead.
This is consistent with the fact that linalg ops deduce their loop bounds from their operands.

Note however that this is not the case for the `tensor<0xelt_type>` which is a special convention
that must be lowered away into either `memref<elt_type>` or just `elt_type` before this
canonicalization can kick in.

Differential Revision: https://reviews.llvm.org/D85413
The file was modifiedmlir/lib/Dialect/Linalg/IR/LinalgOps.cpp
The file was modifiedmlir/test/Dialect/Linalg/canonicalize.mlir
The file was modifiedmlir/lib/IR/StandardTypes.cpp
The file was modifiedmlir/include/mlir/Dialect/Linalg/IR/LinalgStructuredOps.td
Commit 0d33a8ef5bb640ec8b09a41bdc146a556d59c0b7 by paul.walker
[SVE] Lower scalable vector mul operations.

This allows us to remove extra patterns from AArch64SVEInstrInfo.td
because we can reuse those required for fixed length vectors.

Differential Revision: https://reviews.llvm.org/D85328
The file was modifiedllvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
The file was modifiedllvm/test/CodeGen/AArch64/llvm-ir-to-intrinsic.ll
The file was modifiedllvm/lib/Target/AArch64/AArch64SVEInstrInfo.td
The file was modifiedllvm/lib/Target/AArch64/SVEInstrFormats.td
The file was modifiedllvm/lib/Target/AArch64/AArch64ISelLowering.cpp
Commit 13b4db4ec20649e2c32f9bee426c2fc555275d25 by llvm-dev
[X86][SSE] Expose all memory offsets in expand load tests

Since we're messing with individual element loads we need to expose this to show whats going on.

Part of the work to fix the masked_expandload.ll regressions in D66004
The file was modifiedllvm/test/CodeGen/X86/pr39666.ll
The file was modifiedllvm/test/CodeGen/X86/masked_expandload.ll
Commit 87c5437afd273e909e0fed3389de7531d5452ea5 by llvm-dev
PDBExtras.h - remove unnecessary raw_ostream forward declaration. NFCI.

We already need to include raw_ostream.h, also add missing StringRef.h and cstdint implicit dependencies.

Remove unnecessary includes from PDBExtras.cpp
The file was modifiedllvm/include/llvm/DebugInfo/PDB/PDBExtras.h
The file was modifiedllvm/lib/DebugInfo/PDB/PDBExtras.cpp
Commit 807467009d6faa5beb732d49ec4575fa1409abd7 by llvm-dev
[X86] getX86MaskVec - replace mask limit from NumElts < 8 with NumElts <= 4

As noted on PR46885, the number of mask elements should always be a power of 2, so to fix the static analyzer warning we are better off replacing the condition to <= 4, and I've added a pow2 assertion as well.
The file was modifiedllvm/lib/IR/AutoUpgrade.cpp
Commit 55ead5bfffdc00e84cff347ee98471b5616a9f48 by llvm-dev
Fix include sorting order. NFC
The file was modifiedllvm/include/llvm/DebugInfo/PDB/PDBExtras.h
Commit d9e191cb178afd0ab06f4bc932522ddf2e15ac9e by omair.javaid
Revert "[ELF] Allow sections after a non-SHF_ALLOC section to be covered by PT_LOAD"

This reverts commit 030ddc0a0bb9e2b25319eb681d520a9cee32b761.

This breaks http://lab.llvm.org:8011/builders/lldb-arm-ubuntu
and http://lab.llvm.org:8011/builders/lldb-aarch64-ubuntu

Differential Revision: https://reviews.llvm.org/D85100
The file was modifiedlld/test/ELF/linkerscript/symbols-non-alloc.test
The file was modifiedlld/ELF/LinkerScript.cpp
The file was modifiedlld/ELF/Writer.cpp
The file was modifiedlld/test/ELF/linkerscript/sections.s
The file was modifiedlld/test/ELF/linkerscript/compress-debug-sections-custom.s