<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">On Dec 20, 2013, at 4:06 PM, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br><div><blockquote type="cite"><div dir="ltr">+John, since it was his code.</div></blockquote><blockquote type="cite"><div dir="ltr"><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></blockquote><div><br></div>A class should definitely be invalid if any of its base classes is invalid. Making it invalid when one of its members is invalid is probably for the best; I would guess that that’s now occurring through some other channel, causing the original test to successfully not crash.</div><div><br></div><div>I’m not sure what about the linked patch is making a base class invalid when a method in a derived class is invalidated, though?</div><div><br></div><div>John.</div></body></html>