This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] [test] allocator<const T> is non-Standard.
ClosedPublic

Authored by STL_MSFT on Nov 17 2016, 1:34 PM.

Details

Summary

[libcxx] [test] allocator<const T> is non-Standard.

N4582 17.6.3.5 [allocator.requirements] says that allocators are given
cv-unqualified object types, and N4582 20.9.9 [default.allocator]
implies that allocator<const T> is ill-formed (due to colliding
address() overloads). Therefore, tests for allocator<const T>
should be marked as libcxx-specific (if not removed outright).

Diff Detail

Event Timeline

STL_MSFT updated this revision to Diff 78412.Nov 17 2016, 1:34 PM
STL_MSFT retitled this revision from to [libcxx] [test] allocator<const T> is non-Standard..
STL_MSFT updated this object.
STL_MSFT added reviewers: EricWF, mclow.lists.
STL_MSFT added a subscriber: cfe-commits.
EricWF accepted this revision.Nov 17 2016, 11:01 PM
EricWF edited edge metadata.
This revision is now accepted and ready to land.Nov 17 2016, 11:01 PM
STL_MSFT closed this revision.Nov 18 2016, 2:05 PM

Thanks, r287381.