[PATCH] D138899: [PowerPC] add a peephole to eliminate unnecessary load

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 30 06:56:22 PST 2022


nemanjai requested changes to this revision.
nemanjai added a comment.
This revision now requires changes to proceed.

Please add some information about other approaches you considered/attempted to solve this problem. I really don't like the idea of doing store forwarding in the peephole. Are there no existing passes that can perform this type of store forwarding (and most likely more general forms of it)? While I am not all that familiar with `MemorySSA`, it would seem that this is the exact type of thing it should handle effortlessly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138899



More information about the llvm-commits mailing list