This is an archive of the discontinued LLVM Phabricator instance.

[clang] Extract function for generated part of clang::hasAttribute (NFC)
ClosedPublic

Authored by wanders on Feb 20 2023, 8:36 AM.

Details

Summary

This makes it easier to add additional handling when the
tablegen-generated code does not find a match.

No functional change intended.

Diff Detail

Event Timeline

wanders created this revision.Feb 20 2023, 8:36 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 20 2023, 8:36 AM
Herald added a subscriber: jdoerfert. · View Herald Transcript
wanders requested review of this revision.Feb 20 2023, 8:36 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 20 2023, 8:36 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
wanders retitled this revision from [clang] Extract function for generated part of clang::hasAttribute (NFC) r=aaron.ballman to [clang] Extract function for generated part of clang::hasAttribute (NFC).Feb 20 2023, 8:42 AM
erichkeane added inline comments.
clang/lib/Basic/Attributes.cpp
39–41

Why is this not just return hasAttributeImpl(...)?

I see now having seen the other review, LGTM.

erichkeane accepted this revision.Feb 28 2023, 2:39 PM
This revision is now accepted and ready to land.Feb 28 2023, 2:39 PM
wanders updated this revision to Diff 504044.Mar 10 2023, 12:29 AM
wanders edited the summary of this revision. (Show Details)
wanders updated this revision to Diff 504151.Mar 10 2023, 7:50 AM
This revision was landed with ongoing or failed builds.Mar 13 2023, 8:48 AM
This revision was automatically updated to reflect the committed changes.