This is an archive of the discontinued LLVM Phabricator instance.

Implement predefined stdint macros
ClosedPublic

Authored by binji on Jun 13 2014, 2:52 PM.

Details

Summary

Add predefined stdint macros that match the given patterns:

U?INT{_,_FAST,_LEAST}{8,16,32,64}_{MAX,TYPE}
U?INT{PTR,MAX}_{MAX,TYPE}

Diff Detail

Event Timeline

binji updated this revision to Diff 10408.Jun 13 2014, 2:52 PM
binji retitled this revision from to Implement predefined stdint macros.
binji updated this object.
binji edited the test plan for this revision. (Show Details)
binji added reviewers: chandlerc, rnk, rsmith.
binji added a subscriber: Unknown Object (MLST).
rsmith accepted this revision.Jun 24 2014, 9:14 AM
rsmith edited edge metadata.

LGTM, thanks.

We may want to revisit the MSVCCompat check in the future (perhaps we should have a GNU extensions flag that's on by default and controls this), but this seems fine for now.

This revision is now accepted and ready to land.Jun 24 2014, 9:14 AM
jfb closed this revision.Jun 24 2014, 6:41 PM
jfb added a subscriber: jfb.

I ran the tests, everything seems to pass, and landed this for you as r211657.