This is an archive of the discontinued LLVM Phabricator instance.

[clang] Extract ParsedAttrInfo::hasSpelling method (NFC)
ClosedPublic

Authored by wanders on Mar 10 2023, 12:29 AM.

Details

Summary

This intends to simplify this checking when it is done in more places.

No functional change intended.

Diff Detail

Event Timeline

wanders created this revision.Mar 10 2023, 12:29 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 10 2023, 12:29 AM
wanders requested review of this revision.Mar 10 2023, 12:29 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 10 2023, 12:29 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
erichkeane accepted this revision.Mar 10 2023, 6:13 AM

Looks like you have a clang-format fix to do (see pre-commit CI), else, LGTM.

This revision is now accepted and ready to land.Mar 10 2023, 6:13 AM
tbaeder added inline comments.
clang/include/clang/Basic/ParsedAttrInfo.h
92
wanders updated this revision to Diff 504150.Mar 10 2023, 7:50 AM
wanders edited the summary of this revision. (Show Details)
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.