[PATCH] D118160: [RISCV] Implement a basic version of AArch64RedundantCopyElimination pass.

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 3 01:36:32 PST 2022


asb accepted this revision.
asb added a comment.
This revision is now accepted and ready to land.

LGTM. It feels like this could perhaps be made target-independent with some additional hooks - a companion to or modification to isConstantPhysReg that indicates what the constant value is, and using analyzeBranchPredicate. Though extending it in some of the ways suggested in the TODOs would be more difficult, and as the pass itself as quite simple, the code added in terms of target hooks / interface modifications could quickly become larger than the pass itself.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118160



More information about the llvm-commits mailing list