This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] [test] In atomics.types.generic/address.pass.cpp, use T(0) for consistency.
ClosedPublic

Authored by STL_MSFT on Jun 24 2016, 6:33 PM.

Details

Summary

In atomics.types.generic/address.pass.cpp, use T(0) for consistency.

Fixes MSVC "error C2593: 'operator ==' is ambiguous".

There is almost certainly an MSVC compiler bug involved here (which I haven't reduced yet). However, T(0) was used previously in the test, so using it here increases consistency, in addition to making all compilers happy.

Diff Detail

Event Timeline

STL_MSFT updated this revision to Diff 61872.Jun 24 2016, 6:33 PM
STL_MSFT retitled this revision from to [libcxx] [test] In atomics.types.generic/address.pass.cpp, use T(0) for consistency..
STL_MSFT updated this object.
STL_MSFT added reviewers: EricWF, mclow.lists.
STL_MSFT added a subscriber: cfe-commits.
EricWF accepted this revision.Jun 26 2016, 1:03 PM
EricWF edited edge metadata.

Never thrilled working around a compiler bug, but this seems harmless enough.

This revision is now accepted and ready to land.Jun 26 2016, 1:03 PM
EricWF closed this revision.Jun 26 2016, 1:06 PM

r273822.