[PATCH] D59094: [ARM] Fix bug 39982 - pcs("aapcs-vfp") is not consistent
Carey Williams via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Mar 14 09:48:27 PDT 2019
carwil updated this revision to Diff 190653.
carwil marked an inline comment as done.
carwil added a comment.
I got bit a confused earlier. That does actually seem like correct behaviour (once we're no longer able to treat the struct as a homogeneous aggregate).
I've tightened up the conditional a bit as it wouldn't have previously accounted for the case where you had mfloat-abi=hard with an AAPCS (non VFP) attribute. Although in this instance, the backend does seem to be correcting/ignoring that, it's still best we don't blindly rely on such behaviour.
I've also re-structured the conditional and added some comments so it's (hopefully) a little bit clearer what's going on.
I believe this fixes the bug at hand. There's some other calls to getABIKind referencing AAPCS16_VFP, but that doesn't have a direct translation to an LLVM/FI CallingConvention (that I can see). Seems there is a lot of different ways to check various ABI/CC's and none of them seem able to give you the full picture at any point.
CHANGES SINCE LAST ACTION
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 6047 bytes
Desc: not available
More information about the cfe-commits