<div dir="ltr">Hi Warren,<div><br></div><div>I've just been hunting a bug in MS mode where records were being given different layouts depending on if the TU was built with a PCH or not. In the end I tracked it down to <span class="">ASTContext</span><span class="">::</span>getASTRecordLayout() introduced in r192494:</div>








<div><br></div><div><div>  if (isMsLayout(D) && !D->getASTContext().getExternalSource()) {</div><div>    NewEntry = BuildMicrosoftASTRecordLayout(D);</div></div><div><br></div><div>Removing the "&& !D->getASTContext().getExternalSource()" restored the expected behaviour. Is this check still needed?</div>
<div><br></div><div>Cheers,</div><div>Will.</div></div>