[PATCH] D82660: [RISCV] Optimize multiplication by specific immediates

Luís Marques via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 6 12:54:29 PDT 2020


luismarques accepted this revision.
luismarques added a comment.
This revision is now accepted and ready to land.

LGTM.
I'm not overly concerned about the occasional code size increases from doing the optimization for RV32IM, so the loosening of the condition is OK IMO.
Everything else seems to be in order now.
Maybe wait a couple of days more for @lenary's OK.


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

https://reviews.llvm.org/D82660





More information about the llvm-commits mailing list