[PATCH] D117663: [RISCV] Fix the bug in the register allocator caused by reserved BP.

Fraser Cormack via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 20 03:12:54 PST 2022


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

LGTM. You've explained the issue well.



================
Comment at: llvm/test/CodeGen/RISCV/rvv/reg-alloc-reserve-bp.ll:23
 ; CHECK-NEXT:    mv s1, sp
-; CHECK-NEXT:    mv s1, a0
 ; CHECK-NEXT:    vsetivli zero, 8, e32, m2, ta, mu
----------------
kito-cheng wrote:
> Just highlight the code gen issue here, use use `s1`as `BP` but we use `s1`again here, and that what we want to fix by this patch.
Thanks, that's helpful!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117663



More information about the llvm-commits mailing list