[PATCH] D86480: [RISC-V] ADDI/ORI/XORI x, 0 should be as cheap as a move

Luís Marques via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 25 02:28:06 PDT 2020


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

Looking at the AArch64 implementation, it seems there is precedent for this hook already including other instructions that are cheap but not necessarily recognized as actual moves by the microarchitecture (i.e. renames). So I guess they may not be, strictly speaking, as cheap as a move, but they are close enough. And that may very well be the point of the hook, otherwise they would just be non-canonical moves?

In sum, LGTM.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86480



More information about the llvm-commits mailing list