On Mon, Jul 20, 2015 at 1:07 PM, Reid Kleckner <r...@google.com> wrote:
> On Sun, Jul 19, 2015 at 2:41 PM, Richard Smith <richard-l...@metafoo.co.uk
> > wrote:
>
>> @@ -794,8 +795,15 @@ IdentifierInfo *ASTIdentifierLookupTrait
>> // Set or check the various bits in the IdentifierInfo structure.
>> // Token IDs are read-only.
>> if (HasRevertedTokenIDToIdentifier && II->getTokenID() !=
>> tok::identifier)
>> - II->RevertTokenIDToIdentifier();
>> - II->setObjCOrBuiltinID(ObjCOrBuiltinID);
>> + II->revertTokenIDToIdentifier();
>> + if (!F.isModule())
>> + II->setObjCOrBuiltinID(ObjCOrBuiltinID);
>> + else if (HasRevertedBuiltin && II->getBuiltinID()) {
>> + II->revertBuiltin();
>> + assert((II->hasRevertedBuiltin() ||
>> + II->getObjCOrBuiltinID() == ObjCOrBuiltinID) &&
>> + "Incorrect ObjC keyword or builtin ID");
>> + }
>> assert(II->isExtensionToken() == ExtensionToken &&
>> "Incorrect extension token flag");
>> (void)ExtensionToken;
>>
>
> This assert is tripping for me locally on Windows.
>
For posterity: this was caused by a stale modules cache, fixed in r242710.
(It looks like this was only passing elsewhere because it bumps the mtime
on various inputs to the module; that doesn't cause a module rebuild on
Windows.)
_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
