This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Consistently guard "#pragma clang" with "#if defined(__clang__)".
ClosedPublic

Authored by STL_MSFT on Apr 27 2016, 2:17 PM.

Details

Summary

[libc++] Consistently guard "#pragma clang" with "#if defined(clang)".

Similarly, consistently guard "#pragma GCC" with "#if defined(GNUC)".

Fixes MSVC "warning C4068: unknown pragma".

Diff Detail

Event Timeline

STL_MSFT updated this revision to Diff 55311.Apr 27 2016, 2:17 PM
STL_MSFT retitled this revision from to [libc++] Consistently guard "#pragma clang" with "#if defined(__clang__)"..
STL_MSFT updated this object.
STL_MSFT added reviewers: mclow.lists, EricWF.
STL_MSFT added a subscriber: cfe-commits.
EricWF accepted this revision.Apr 27 2016, 6:50 PM
EricWF edited edge metadata.
This revision is now accepted and ready to land.Apr 27 2016, 6:50 PM
EricWF closed this revision.Apr 27 2016, 6:55 PM

Committed in r267836. I must say it's pretty cool to review and commit a patch from *the* STL. Never thought that would happen when I started programming.