[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