erichkeane added a comment.

This is my concern here: 
https://godbolt.org/z/icS4fa
The patch will change template instantiation.

GCC doesn't seem to allow using _Atomic in C++ mode, which is perhaps a 
necessary part of this solution?  I see we already consider overload sets with 
int and _Atomic(int) to be ambiguous (in addition to not considering conversion 
operators of _Atomic(int) at all??), so it is limited to type deduction. I 
could see this causing problems, since specializations of templates are allowed 
on int and _Atomic(int) despite not being used in overload sets; 
https://godbolt.org/z/oMAvpz


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59307/new/

https://reviews.llvm.org/D59307



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to