<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Aug 1, 2014 at 12:14 PM, jahanian <span dir="ltr"><<a href="mailto:fjahanian@apple.com" target="_blank">fjahanian@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div class=""><br><div><div>On Aug 1, 2014, at 11:44 AM, Reid Kleckner <<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>> wrote:</div>
<br><blockquote type="cite"><div dir="ltr">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.</div>
<div class="gmail_extra">
<br></div></blockquote></div><br></div><div>I first thought I could use AlignmentIsRequired. But then I noticed that AlignmentIsRequired  is set only if current typede<font color="#4f8187" face="Menlo"><span style="font-size:11px">f has explicit</span></font></div>
<div><font color="#4f8187" face="Menlo"><span style="font-size:11px">alignment. (see Decl::getMaxAlignment). </span></font><span style="font-family:Menlo;font-size:11px">We want to make sure that a typedef of an aligned typedef is </span></div>
<div><span style="font-family:Menlo;font-size:11px">still considered explicitly aligned (and not to modify its alignment). My patch has a test for this case.</span></div></div></blockquote><div><br></div><div>That sounds like a bug.</div>
</div></div></div>