[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
Wed Jan 19 06:17:15 PST 2022
frasercrmck added inline comments.
================
Comment at: llvm/lib/Target/RISCV/RISCVFrameLowering.cpp:959
+ // Refer to https://github.com/llvm/llvm-project/issues/53016.
+ return MF.getSubtarget<RISCVSubtarget>().hasStdExtV();
}
----------------
kito-cheng wrote:
> Maybe `MF.getSubtarget<RISCVSubtarget>().hasVInstructions()`? although that same as `hasStdExtV` for now, but that prevent we need to check `zve*` ext. here in future.
Yeah, I think that would be best.
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