This is an archive of the discontinued LLVM Phabricator instance.

[clang] Add missing header guard in <cpuid.h>
ClosedPublic

Authored by mcgrathr on Nov 10 2020, 6:57 PM.

Details

Summary

This header has long lacked a standard multiple inclusion guard
like other headers have, for no apparent reason. The GCC header
of the same name likewise lacks one up through release 10.1, but
trunk GCC (release 11, and perhaps future 10.x) has fixed it
(see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96238).

Diff Detail

Event Timeline

mcgrathr created this revision.Nov 10 2020, 6:57 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 10 2020, 6:57 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
mcgrathr requested review of this revision.Nov 10 2020, 6:57 PM
phosek accepted this revision.Nov 10 2020, 7:31 PM

LGTM

This revision is now accepted and ready to land.Nov 10 2020, 7:31 PM
This revision was landed with ongoing or failed builds.Nov 10 2020, 7:34 PM
This revision was automatically updated to reflect the committed changes.