Patch to force SuitableAlign's alignment when loading on object with larger alignment

jahanian fjahanian at apple.com
Fri Aug 1 12:14:23 PDT 2014


On Aug 1, 2014, at 11:44 AM, Reid Kleckner <rnk at google.com> wrote:

> What is HasAlignAttribute, and how is it different from AlignmentIsRequired? I have a feeling that they are different names for the same thing. If you want to rename it, that's fine with me.
> 

I first thought I could use AlignmentIsRequired. But then I noticed that AlignmentIsRequired  is set only if current typedef has explicit
alignment. (see Decl::getMaxAlignment). We want to make sure that a typedef of an aligned typedef is 
still considered explicitly aligned (and not to modify its alignment). My patch has a test for this case.
- Fariborz

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140801/55975125/attachment.html>


More information about the cfe-commits mailing list