<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Aug 1, 2014 at 12:43 PM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5">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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><div><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></div><div>
That sounds like a bug.</div>
</div></div></div>
</blockquote></div><br></div><div class="gmail_extra">This bug should be fixed in r214698.</div></div>