This is an archive of the discontinued LLVM Phabricator instance.

Assume existence of inttypes.h and stdint.h in DataTypes.h.
ClosedPublic

Authored by thakis on Apr 1 2018, 5:30 PM.

Details

Reviewers
rnk
deadalnix
Summary

These should exist in all toolchains LLVM supports nowadays.

Enables making DataTypes.h a regular header instead of a .h.cmake file and allows deleting a bunch of cmake goop (which should also speed up cmake configure time a bit).

All the code this removes is 9+ years old.

Diff Detail

Event Timeline

thakis created this revision.Apr 1 2018, 5:30 PM
thakis updated this revision to Diff 140605.Apr 1 2018, 6:08 PM
rnk added a comment.Apr 1 2018, 8:26 PM

You should remove the checks for these headers in config-ix.cmake, that will actually remove the configure-time compile test for the existence of these headers.

rnk accepted this revision.Apr 1 2018, 8:27 PM

Ignore that, I just missed it on the first pass.

This revision is now accepted and ready to land.Apr 1 2018, 8:27 PM
thakis closed this revision.Apr 2 2018, 6:25 AM

r328970, thanks!

include/llvm-c/DataTypes.h