pratlucas marked 2 inline comments as done.
pratlucas added inline comments.
================
Comment at: llvm/lib/Target/ARM/ARMFrameLowering.cpp:732
+ // AAPCS requires use of R11, and PACBTI gets in the way of regular pushes,
+ // so FP ends up on area two.
if (HasFP) {
----------------
efriedma wrote:
> I guess this is related to this patch because it involves the interaction of
> PACBTI with -mframe_chain ? (Without this patch, PACBTI doesn't exist on any
> targets where the frame pointer is in r11.) I'm fine leaving it to a
> followup, though.
Sounds good. I'm going to address that on a followup patch.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125094/new/
https://reviews.llvm.org/D125094
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
