[moving to cfe-commits]

> I think that serialization is automatically handled for all options
> defined in LangOptions.def (ASTWriter::WriteControlBlock).  But
> because this is a BENIGN_LANGOPT, ASTReader seems to skip checking
> that they match.

Hmm, I was worried about changing this option from BENIGN_LANGOPT to LANGOPT 
because it would affect other code, but now that I look at it more closely it 
seems to do just what I want.   Additionally langopt also adds this flag to 
-module-file-info, which seems like a good thing.  I’ve attached an updated 
patch that switches to LANGOPT.

Attachment: module-system-header-comments.patch
Description: Binary data


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

Reply via email to