This is an archive of the discontinued LLVM Phabricator instance.

Treat inttypes.h as a built-in header
Needs ReviewPublic

Authored by markrowe on Aug 2 2021, 10:46 AM.

Details

Summary

It wraps a system header. This allows the system module map to declare a
module for it.

https://bugs.llvm.org/show_bug.cgi?id=51307

Diff Detail

Event Timeline

markrowe requested review of this revision.Aug 2 2021, 10:46 AM
markrowe created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptAug 2 2021, 10:46 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
dexonsmith resigned from this revision.Aug 16 2021, 3:55 PM
dexonsmith added reviewers: bruno, vsapsai.
dexonsmith added subscribers: bruno, vsapsai.

@bruno or @vsapsai, any chance one of you knows whether this is correct?

In general it is correct but there is no test case. I have a test case but it fails for other reasons. And I wasn't able to find time to address these "other reasons". I'll post my version of the test case soon.