FailedChanges

Summary

  1. [mlir] Fix ssa values naming bug (details)
  2. Optimize GSymCreator::finalize. (details)
  3. Change the context instruction for computeKnownBits in LoadStoreVectorizer pass (details)
  4. [mlir][Linalg] Add interface methods to get lhs and rhs of contraction (details)
Commit fb3a00c327df78eaa534e53ac6f07112e0585121 by riddleriver
[mlir] Fix ssa values naming bug

Address comments in https://reviews.llvm.org/D102226 to fix the bug + style violations

Differential Revision: https://reviews.llvm.org/D102368
The file was modifiedmlir/lib/IR/AsmPrinter.cpp
Commit e5bdacba2e185034979fddd8bff2695bfcdd3056 by gclayton
Optimize GSymCreator::finalize.

The algorithm removing duplicates from the Funcs list used to have
amortized quadratic time complexity because it was potentially
removing each entry using std::vector::erase individually. This
patch is now using a erase-remove idiom with an adapted
removeIfBinary algorithm.

Probably this was made under the assumption that these removals are
rare, but there are cases where the case of duplicate entries is
occurring frequently. In these cases, the actual runtime was very
poor, taking hours to process a single binary of around 1 GiB size
including debug info. Another factor contributing to that is the
frequent output of the warning, which is now removed.

It seems this is particularly an issue with GCC-compiled binaries,
rather than clang-built binaries.

Reviewed By: clayborg

Differential Revision: https://reviews.llvm.org/D102219
The file was modifiedllvm/lib/DebugInfo/GSYM/GsymCreator.cpp
Commit e7d26aceca071d67168062b2f7784c56234b0cb3 by Justin Bogner
Change the context instruction for computeKnownBits in LoadStoreVectorizer pass

This change enables cases for which the index value for the first
load/store instruction in a pair could be a function argument. This
allows using llvm.assume to provide known bits information in such
cases.

Patch by Viacheslav Nikolaev. Thanks!

Differential Revision: https://reviews.llvm.org/D101680
The file was modifiedllvm/lib/Transforms/Vectorize/LoadStoreVectorizer.cpp
The file was modifiedllvm/test/Transforms/LoadStoreVectorizer/X86/vectorize-i8-nested-add.ll
Commit b7911e80d6926f9280ceb23d4e86e25c29370904 by ravishankarm
[mlir][Linalg] Add interface methods to get lhs and rhs of contraction

Differential Revision: https://reviews.llvm.org/D102301
The file was modifiedmlir/include/mlir/Dialect/Linalg/IR/LinalgInterfaces.td