This is an archive of the discontinued LLVM Phabricator instance.

[Lex] Consider a PCH header-guarded even with #endif truncated
ClosedPublic

Authored by sammccall on Jul 16 2021, 5:29 PM.

Details

Summary

This seems to be a more useful behavior for tools that use preambles.
I believe it doesn't affect real compiles: the PCH is only included once
when used, and recursive inclusion of the main-file *within* the PCH
isn't supported in any case.

Diff Detail

Event Timeline

sammccall created this revision.Jul 16 2021, 5:29 PM
sammccall requested review of this revision.Jul 16 2021, 5:29 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJul 16 2021, 5:29 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
kadircet accepted this revision.Jul 19 2021, 10:07 AM
This revision is now accepted and ready to land.Jul 19 2021, 10:07 AM
This revision was landed with ongoing or failed builds.Jul 20 2021, 5:37 AM
This revision was automatically updated to reflect the committed changes.