[PATCH] D138876: [RISCV] Share code for fixed offsets adjustRegs (thus materializing fewer constants)

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 30 11:01:01 PST 2022


reames closed this revision.
reames added a comment.

This has landed, but I accidentally tagged with the wrong review so auto-close didn't catch it.

commit ac1ec9e2904a696e360b40572c3b3c29d67981ef <https://reviews.llvm.org/rGac1ec9e2904a696e360b40572c3b3c29d67981ef>
Author: Philip Reames <preames at rivosinc.com>
Date:   Wed Nov 30 07:22:52 2022 -0800

  [RISCV] Share code for fixed offsets adjustRegs (thus materializing fewer constants)
  
  This reuses the existing optimized implementation of adjustReg, and commons up code. This has the effect of enabling two code changes for the new caller. First, we enable the "split andi" lowering (with no alignment requirement), and second we use a sub with smaller constant in register instead of a add with negative constant in register.
  
  Differential Revision: https://reviews.llvm.org/D132839


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

https://reviews.llvm.org/D138876



More information about the llvm-commits mailing list