[PATCH] D134129: [PATCH] [RISCV] Enable -msave-restore by default when optimizing for size
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 8 07:45:24 PDT 2023
craig.topper added a comment.
In D134129#4326811 <https://reviews.llvm.org/D134129#4326811>, @hiraditya wrote:
> In D134129#4191314 <https://reviews.llvm.org/D134129#4191314>, @craig.topper wrote:
>
>> Would this break shadow call stack which isn't compatible with save/restore lib calls? Or do we already check that somewhere?
>
> Will this still be the issue when X3 will be used for SCS?
I don't think so as long as the error about mixing -msave-restore with SCS was removed from RISCVFrameLowering.cpp when we switched to X3.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134129/new/
https://reviews.llvm.org/D134129
More information about the llvm-commits
mailing list