[PATCH] D79118: Implement _ExtInt ABI for all ABIs in Clang, enable type for ABIs
John McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon May 4 12:54:28 PDT 2020
rjmccall added inline comments.
================
Comment at: clang/lib/CodeGen/TargetInfo.cpp:4938
+ if (EIT->getNumBits() > 128)
+ return getNaturalAlignIndirect(Ty, /*ByVal=*/true);
+
----------------
erichkeane wrote:
> rjmccall wrote:
> > Does this need to consider the aggregate-as-array logic below?
> I'm not sure what you mean by this? Are you suggesting I could/should pass this as an array instead of indirectly?
My interpretation is that in general you're lowering large a `_ExtInt` like a struct with a bunch of integer members in it. My understanding is that, for this target, that would make it a homogeneous aggregate eligible for the special treatment given to certain aggregate types below. I don't know what that corresponds to at the code-emission level.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79118/new/
https://reviews.llvm.org/D79118
More information about the cfe-commits
mailing list