[PATCH] D105417: [RISCV] Teach RISCVMatInt about cases where it can use LUI+SLLI to replace LUI+ADDI+SLLI for large constants.

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

luismarques added a comment.

In D105417#2874534 <https://reviews.llvm.org/D105417#2874534>, @craig.topper wrote:

> @luismarques were you able to find the change you were looking at to fix the add-before-shl.ll regression?

I just checked that I indeed have a commit where that isn't an issue, but it's not a localized fix for that regression. I was computing the cost of instruction sequences and backing off if they were more "costly" (based on instruction count and length). I was working part-time these last two weeks, so I didn't get around to properly dig that work up, refresh it and share it, but I'll try to do it by ~Sunday.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105417