This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Disable pragma system_header in the new testing configuration
ClosedPublic

Authored by ldionne on Jul 16 2021, 1:44 PM.

Details

Summary

The new testing configuration did not turn off #pragma system_header,
which means we were not seeing warnings in system headers.

I expect this to start failing on GCC (see the removed comment), but
I'd like to fix the root cause before I ship this.

Diff Detail

Event Timeline

ldionne created this revision.Jul 16 2021, 1:44 PM
ldionne requested review of this revision.Jul 16 2021, 1:44 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 16 2021, 1:44 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne updated this revision to Diff 359793.Jul 19 2021, 7:57 AM

Fix a bunch of warnings - I expect there are still some issues.

ldionne updated this revision to Diff 359802.Jul 19 2021, 8:24 AM

Don't enable warnings in system headers for GCC - there's too many to fix for now.

ldionne accepted this revision.Jul 19 2021, 11:38 AM
This revision is now accepted and ready to land.Jul 19 2021, 11:38 AM