Hi Warren,

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 ASTContext::getASTRecordLayout() introduced in
r192494:

  if (isMsLayout(D) && !D->getASTContext().getExternalSource()) {
    NewEntry = BuildMicrosoftASTRecordLayout(D);

Removing the "&& !D->getASTContext().getExternalSource()" restored the
expected behaviour. Is this check still needed?

Cheers,
Will.

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to