This is an archive of the discontinued LLVM Phabricator instance.

[libcxx][tests] span: Add failing tests for span::first and span::last
ClosedPublic

Authored by miscco on Dec 30 2019, 5:30 AM.

Details

Summary

Both methods have compile time constraints that we should test against

Diff Detail

Event Timeline

miscco created this revision.Dec 30 2019, 5:30 AM
Herald added a project: Restricted Project. · View Herald Transcript
miscco updated this revision to Diff 235590.Dec 30 2019, 5:46 AM

Updating D71999: [libcxx][tests] span: Add failing tests for span::first and span::last

miscco updated this revision to Diff 235591.Dec 30 2019, 5:48 AM

Updating D71999: [libcxx][tests] span: Add failing tests for span::first and span::last

Harbormaster completed remote builds in B43044: Diff 235591.
ldionne accepted this revision.Feb 10 2020, 4:53 AM

Thanks for the patch!

This revision is now accepted and ready to land.Feb 10 2020, 4:53 AM
This revision was automatically updated to reflect the committed changes.