This is an archive of the discontinued LLVM Phabricator instance.

[libc++][test] Fix max_size test for Arm big endian
ClosedPublic

Authored by michaelplatings on Aug 11 2023, 3:11 AM.

Details

Reviewers
philnik
Group Reviewers
Restricted Project
Commits
rGd13f1cb569ca: [libc++][test] Fix max_size test for Arm big endian
Summary

With ABI version 2 and big endian Arm, the full_size test matches the
current behaviour. It's possible that this would also be the case for
other big endian architectures but I don't have data on that.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2023, 3:11 AM
michaelplatings requested review of this revision.Aug 11 2023, 3:11 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2023, 3:11 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
michaelplatings retitled this revision from [libc++][test] Fix max_test for Arm big endian to [libc++][test] Fix max_size for Arm big endian.Aug 11 2023, 3:13 AM
michaelplatings retitled this revision from [libc++][test] Fix max_size for Arm big endian to [libc++][test] Fix max_size test for Arm big endian.Aug 11 2023, 3:15 AM
philnik accepted this revision.Aug 11 2023, 8:06 AM
This revision is now accepted and ready to land.Aug 11 2023, 8:06 AM
This revision was landed with ongoing or failed builds.Aug 11 2023, 8:25 AM
This revision was automatically updated to reflect the committed changes.