This is an archive of the discontinued LLVM Phabricator instance.

[libc++][NFC] Enable modernize-use-override
ClosedPublic

Authored by philnik on Apr 30 2022, 6:52 AM.

Details

Diff Detail

Event Timeline

philnik created this revision.Apr 30 2022, 6:52 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 30 2022, 6:52 AM
Herald added a subscriber: smeenai. · View Herald Transcript
philnik requested review of this revision.Apr 30 2022, 6:52 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 30 2022, 6:52 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript

Nice improvement! Looking at the UBSAN CI failures there are more overrides to be added.
Can you add an NFC to the title of the patch?

libcxx/include/__memory/shared_ptr.h
134

Just curious did you use -fix for clang-tidy?

libcxx/include/codecvt
226–238

Did the indention change?

262–273

Did the indention change?

297–308

Did the indention change?

331–342

Did the indention change?

365–376

Did the indention change?

399–410

Did the indention change? Please also validate similar changes.

philnik updated this revision to Diff 427099.May 4 2022, 11:47 AM
  • Add a few more overrides
philnik retitled this revision from [libc++] Enable modernize-use-override to [libc++][NFC] Enable modernize-use-override.Aug 23 2022, 4:52 PM
philnik updated this revision to Diff 455025.Aug 23 2022, 5:14 PM
philnik marked 7 inline comments as done.
  • Try to fix CI
libcxx/include/__memory/shared_ptr.h
134

Kind-of. I went trough the errors and let my IDE auto-fix them. Using -fix doesn't work very well because we copy the headers around.

libcxx/include/codecvt
226–238

The return type also doesn't have it's own line anymore. That's why the diff looks so weird.

262–273

It changed the same as the last one.

297–308

Same

331–342

same

365–376

same

399–410

same

Mordante accepted this revision.Aug 25 2022, 9:46 AM

LGTM!

This revision is now accepted and ready to land.Aug 25 2022, 9:46 AM
This revision was automatically updated to reflect the committed changes.