[PATCH] D33021: [libcxx] [test] libc++ test changes for CWG 2094

Billy Robert O'Neal III via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 10 10:00:06 PDT 2017


BillyONeal added a comment.

In https://reviews.llvm.org/D33021#750831, @mclow.lists wrote:

> FWIW, I think this is a language change that core got wrong, and it has already come back to bite us. (https://bugs.llvm.org/show_bug.cgi?id=28527)


Hmm.... we actually took steps to do the opposite in std::copy -- to use memcpy even in the presence of volatile -- as a result of this change.

It looks like Core wasn't a fan of doing this either, but that the Itanium ABI forced their hand.


https://reviews.llvm.org/D33021





More information about the cfe-commits mailing list