<div dir="ltr">Committed as r201810.<br><div><br></div><div>-Warren</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Feb 20, 2014 at 12:15 PM, Warren Hunt <span dir="ltr"><<a href="mailto:whunt@google.com" target="_blank">whunt@google.com</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">After further discussion/investigation, the check is wrong and we should remove it.  Without the check, if we are debugging on windows using lldb and DWARF, we can potentially get the wrong layout during a debugging session in which someone defines a new type.  Given that don't have support debugging on windows using lldb, this is fine.  I'll patch it.<span class="HOEnZb"><font color="#888888"><div>

<br></div><div>-Warren</div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Feb 20, 2014 at 6:16 AM, Will Wilson <span dir="ltr"><<a href="mailto:will@indefiant.com" target="_blank">will@indefiant.com</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 Warren,<div><br></div><div>As far as I understand, a PCH is treated much like a simplified clang module and as such any PCH is an ExternalSource meaning the ExternalSource is valid in this case. I don't thing DWARF has any relevance here AFAIK.</div>


<div><br></div><div>It's likely that I was seeing this "delayed" layout behaviour due to the record in question being a class template declared in the PCH that wasn't instantiated in the PCH but later in a compiland including the PCH. After all you can't do record layout for a template class that hasn't yet been instantiated (well unless all the data members weren't dependent on template arguments). So even if known record layout are serialized in the PCH (I don't actually know if they are or not TBH) there will always be class templates instantiations that would have to be laid out after the creation of the PCH - as is the case for me.</div>


<div><br></div><div>Either way, in this case the <span style="font-size:13px;font-family:arial,sans-serif">"&& !D->getASTContext().</span><span style="font-size:13px;font-family:arial,sans-serif">getExternalSource()" still does a layout, it just does one without using MS layout rules.</span></div>

<span><font color="#888888">
<div><br></div><div>- Will.</div>
</font></span></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>