This is an archive of the discontinued LLVM Phabricator instance.

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

Authored by philnik on Aug 11 2023, 10:36 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 depedent return type is shorter.

Diff Detail

Event Timeline

philnik created this revision.Aug 11 2023, 10:36 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2023, 10:36 AM
philnik requested review of this revision.Aug 11 2023, 10:36 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2023, 10:36 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
philnik updated this revision to Diff 549481.Aug 11 2023, 12:00 PM

Fix formatting

philnik updated this revision to Diff 549619.Aug 12 2023, 9:06 AM

Try to fix CI

philnik updated this revision to Diff 550150.Aug 14 2023, 5:15 PM

Fix some _LIBCPP_INLINE_VISIBILITY posisitions

ldionne accepted this revision.Aug 15 2023, 8:30 AM
ldionne added a subscriber: ldionne.

LGTM. CI is green, it's just AIX.

This revision is now accepted and ready to land.Aug 15 2023, 8:30 AM
This revision was landed with ongoing or failed builds.Aug 15 2023, 12:19 PM
This revision was automatically updated to reflect the committed changes.