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

[Luís Marques via Phabricator via llvm-commits]

luismarques added a comment.

This is technically correct, but looking at the other targets it seems that the hook is generally being used more restrictively for instructions where you actually have some expectation of it being a copy/move. Do we really want to go down this route? In principle we'd have to later expand this to e.g. all of the bitmanip instructions that can implement copies and so on, which seems a bit like a reductio ad absurdum. Having tests showing at least a little bit of benefit would help make the case for this patch :-)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86522