[PATCH] D98101: [RISCV] Enable the LocalStackSlotAllocation pass support

Jessica Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 6 18:06:16 PST 2023


jrtc27 added a comment.

In D98101#4032848 <https://reviews.llvm.org/D98101#4032848>, @LiDongjin wrote:

> modify the isFrameOffsetLegal to (Offset <= maxIntN(12)) && (Offset >= minIntN(12)),

That’s what isIntN<12>(Offset) does

> Simply constrain offset to be within the range of 12-bit unsigned integers.

Signed, not unsigned, which is already done


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98101



More information about the llvm-commits mailing list