This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Remove _LIBCPP_HAS_NO_IS_AGGREGATE
ClosedPublic

Authored by ldionne on Aug 10 2021, 8:33 AM.

Details

Reviewers
Mordante
Group Reviewers
Restricted Project
Commits
rG7be03cc782d0: [libc++] Remove _LIBCPP_HAS_NO_IS_AGGREGATE
Summary

All supported compilers have been supporting __is_aggregate for a long
time now, so it's reasonable to remove this workaround.

Diff Detail

Event Timeline

ldionne created this revision.Aug 10 2021, 8:33 AM
ldionne requested review of this revision.Aug 10 2021, 8:33 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 10 2021, 8:33 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Mordante accepted this revision as: Mordante.Aug 10 2021, 8:57 AM

LTGM, provided the build passes.

ldionne updated this revision to Diff 365520.Aug 10 2021, 9:44 AM

Rebase onto main and re-apply patch to try and fix the patch application failure.

ldionne accepted this revision as: Restricted Project.Aug 11 2021, 7:08 AM
This revision is now accepted and ready to land.Aug 11 2021, 7:08 AM
This revision was automatically updated to reflect the committed changes.
libcxx/include/version