[PATCH] shared_ptr : atomic ref-counts (TSan)

Eric Fiselier eric at efcs.ca
Thu May 21 19:54:08 PDT 2015


I think a much safer change would be to use the `__atomic_foo` builtins. It means we wouldn't have to change the type stored in `__shared_count` and `__shared_weak_count` and we wouldn't  depend on the `<atomic>` header.

Is there a reason this approach was not taken?


http://reviews.llvm.org/D8147

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list