This is an archive of the discontinued LLVM Phabricator instance.

[libc++][NFC] Update the remaining enable_ifs
ClosedPublic

Authored by philnik on Aug 11 2023, 1:06 PM.

Details

Reviewers
ldionne
Group Reviewers
Restricted Project
Commits
rG6256ccfd4f06: [libc++][NFC] Update the remaining enable_ifs
Summary

This brings most of the enable_ifs in libc++ to the same style.

Diff Detail

Event Timeline

philnik created this revision.Aug 11 2023, 1:06 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2023, 1:06 PM
philnik requested review of this revision.Aug 11 2023, 1:06 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2023, 1:06 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
ldionne accepted this revision.Sep 1 2023, 8:45 AM
ldionne added a subscriber: ldionne.

LGTM with whitespace nitpick. Also please ensure the CI is green.

libcxx/include/__type_traits/invoke.h
243

Here and below.

This revision is now accepted and ready to land.Sep 1 2023, 8:45 AM
philnik updated this revision to Diff 555414.Sep 1 2023, 9:31 AM
philnik marked an inline comment as done.

Address comments

This revision was automatically updated to reflect the committed changes.