[PATCH] D19758: [libcxx] [test] Replace non-Standard "atomic_flag f(false); " with Standard "atomic_flag f; "

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Mon May 2 17:05:35 PDT 2016


EricWF retitled this revision from "[libcxx] [test] Replace non-Standard "atomic_flag f(false);" with Standard "atomic_flag f = ATOMIC_FLAG_INIT;"." to "[libcxx] [test] Replace non-Standard "atomic_flag f(false);" with Standard "atomic_flag f;" ".
EricWF updated the summary for this revision.
EricWF updated this revision to Diff 55920.
EricWF added a comment.

@STL_MSFT Does this work for you? I simply removed the initializer and call the default constructor instead. This leaves 'f' in an unspecified state but that shouldn't matter because we always call 'f.test_and_set()' before testing the clear operation.


http://reviews.llvm.org/D19758

Files:
  include/atomic
  test/libcxx/atomics/atomics.flag/init_bool.pass.cpp
  test/std/atomics/atomics.flag/atomic_flag_clear.pass.cpp
  test/std/atomics/atomics.flag/atomic_flag_clear_explicit.pass.cpp
  test/std/atomics/atomics.flag/clear.pass.cpp
  test/std/atomics/atomics.flag/init03.pass.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19758.55920.patch
Type: text/x-patch
Size: 6299 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160503/1ab7bb6e/attachment.bin>


More information about the cfe-commits mailing list