This is an archive of the discontinued LLVM Phabricator instance.

[llvm] Fix include guards
ClosedPublic

Authored by IncludeGuardian on May 13 2023, 12:25 PM.

Details

Summary

Add missing include guards to LLVM header files that did not previously
have them and update existing include guards to ensure that they enclose
all non-whitespace, non-comment text to enable these headers for the
multiple-include optimization.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptMay 13 2023, 12:25 PM
IncludeGuardian requested review of this revision.May 13 2023, 12:25 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 13 2023, 12:25 PM
foad added a comment.May 15 2023, 2:42 AM

AMDGPU parts look fine, thanks.

Anyone want to give this a final look and approval?

barannikov88 accepted this revision.Jun 12 2023, 9:24 AM
barannikov88 added a subscriber: barannikov88.

LG, thanks

This revision is now accepted and ready to land.Jun 12 2023, 9:24 AM
lattner accepted this revision.Jun 12 2023, 1:52 PM

cool thx

MaskRay accepted this revision.Jun 12 2023, 1:58 PM
This revision was automatically updated to reflect the committed changes.