<div dir="ltr">+John, since it was his code.<br><div><br></div><div>Are there other ways we can get an invalid record decl and then do layout on a derived class to trigger <span style="font-family:arial,sans-serif;font-size:13px">PR18284?  If so, it seems like this doesn't fix the underlying issue.</span></div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Dec 20, 2013 at 3:59 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>if a method decl is invalid, clang currently sometimes marks the method's record decl invalid, and sometimes doesn't (and this ends up confusing record layout). For consistency, and to fix PR18284, never do this.</div>

<div><br></div><div>(The code to do this was added in <a href="http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20100809/033154.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20100809/033154.html</a> , but the test from that revision is still passing.)</div>

<div><br></div><div>See also this thread <a href="http://lists.cs.uiuc.edu/pipermail/cfe-dev/2013-December/034216.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/cfe-dev/2013-December/034216.html</a> for some discussion.</div>
<span class="HOEnZb"><font color="#888888"><div>
<br></div><div>Nico</div></font></span></div>
</blockquote></div><br></div>