[PATCH] D140046: [PowerPC] Fix up memory ordering after combining BV to a load

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 15 13:59:04 PST 2022


nemanjai updated this revision to Diff 483329.
nemanjai added a comment.

Addressed review comments:

- Switch from `SmallPtrSet` to `SmallVector`
- Used `areNonVolatileConsecutiveLoads()` and updated it to consider the memory VT vs. the loaded VT
- Switched `dyn_cast` to `cast` for converting nodes whose opcodes were already checked


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D140046/new/

https://reviews.llvm.org/D140046

Files:
  llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
  llvm/lib/Target/PowerPC/PPCISelLowering.cpp
  llvm/test/CodeGen/PowerPC/build-vector-to-ld-chain.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D140046.483329.patch
Type: text/x-patch
Size: 6822 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221215/c6db4470/attachment.bin>


More information about the llvm-commits mailing list