[PATCH] D46182: [RISCV] Set isReMaterializable on ADDI and LUI instructions

[Alex Bradbury via Phabricator via llvm-commits]

asb added a comment.

In https://reviews.llvm.org/D46182#1081488, @mgrang wrote:

> @asb Thanks for the patch. I tested this on our internal workload and it gave us ~44 bytes savings.
>  However, instead if we mark all ALU insts as isReMaterializable then we get ~226 bytes savings:
>  Note: Marking ALU insts in addition to ADDI and LUI as isReMaterializable still gave us only ~44 bytes savings.


Hi Mandeep - thanks for taking a look. I'm a bit confused, which of the above 3 statements is true?


Repository:
  rL LLVM

https://reviews.llvm.org/D46182