This is an archive of the discontinued LLVM Phabricator instance.

[libc++][NFC] Refactor enable_ifs in defaulted arguments to defaulted template arguments
ClosedPublic

Authored by philnik on Aug 11 2023, 11:57 AM.

Details

Summary

This brings most of the enable_ifs in libc++ to the same style. It also has the nice side-effect of reducing the size of names of these symbols, since the arguments don't get mangled anymore.

Diff Detail

Event Timeline

philnik created this revision.Aug 11 2023, 11:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2023, 11:57 AM
philnik requested review of this revision.Aug 11 2023, 11:57 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2023, 11:57 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Mordante accepted this revision.Aug 16 2023, 10:06 AM
Mordante added a subscriber: Mordante.

Thanks for the cleanup! When the CI is happy I'm happy.

This revision is now accepted and ready to land.Aug 16 2023, 10:06 AM
philnik updated this revision to Diff 551263.Aug 17 2023, 2:13 PM

Fix formatting