rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
LGTM.
Comment at: clang/lib/CodeGen/CGCall.cpp:2258
Attrs.addDereferenceableAttr(info.first.getQuantity());
-Attrs.addAttribute(llvm::Attribute::getWithA
efriedma marked 2 inline comments as done.
efriedma added inline comments.
Comment at: clang/lib/CodeGen/CGCall.cpp:2116
+if (PTy->isObjectType()) {
+ if (unsigned Alignment = getContext().getTypeAlignIfKnown(PTy))
+RetAttrs.addAlignmentAttr(
rjm
rjmccall added a comment.
Herald added a subscriber: sstefan1.
Hmm. It looks like we did reserve space for this in the great
pointer-alignment-assumption detente ("RFC: Enforcing pointer type alignment in
Clang" from early 2016, which I'm aware was never really a consensus position,
but which
