[PATCH] D86522: [RISC-V] Implement RISCVInstrInfo::isCopyInstrImpl()

Alexander Richardson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 17 01:57:21 PDT 2020


arichardson added a comment.

In D86522#2278710 <https://reviews.llvm.org/D86522#2278710>, @luismarques wrote:

> I was hoping others would chime in regarding what instructions should be handled in this hook.
>
> @arichardson How do you feel about removing the ORI/XORI and landing this patch? (I also have no major objections to just landing as is). We can always further improve it later. Ideally this would add tests, but if that's not easy I think it's OK to land it as is.

Sure, will do that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86522



More information about the llvm-commits mailing list