This is an archive of the discontinued LLVM Phabricator instance.

[libc++][test] Selects proper C++23 field.
ClosedPublic

Authored by Mordante on Apr 30 2023, 12:25 PM.

Details

Reviewers
philnik
Group Reviewers
Restricted Project
Commits
rGcc3be76bea66: [libc++][test] Selects proper C++23 field.
Summary

D149553 changes the name of the LangOptions member. This change allows
libc++ to work with either name.

Diff Detail

Event Timeline

Mordante created this revision.Apr 30 2023, 12:25 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 30 2023, 12:25 PM
Herald added a subscriber: jdoerfert. · View Herald Transcript
Mordante requested review of this revision.Apr 30 2023, 12:25 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 30 2023, 12:25 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
philnik accepted this revision.Apr 30 2023, 12:36 PM

LGTM % nit.

libcxx/test/tools/clang_tidy_checks/uglify_attributes.cpp
32–44

This should also work.

This revision is now accepted and ready to land.Apr 30 2023, 12:36 PM
Mordante updated this revision to Diff 519505.May 4 2023, 8:09 AM

Rebased and addresses review comment.

Mordante marked an inline comment as done.May 4 2023, 8:50 AM

Thanks for the review!

libcxx/test/tools/clang_tidy_checks/uglify_attributes.cpp
32–44

I expected a simpler solution was indeed possible. Thanks for the suggestion.

This revision was landed with ongoing or failed builds.May 4 2023, 10:51 AM
This revision was automatically updated to reflect the committed changes.
Mordante marked an inline comment as done.