mibintc added a comment. In D59307#1427853 <https://reviews.llvm.org/D59307#1427853>, @erichkeane wrote:
> 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 Recognizing _Atomic in c++ mode is an extension right? Probably it needs more thought, for example the test case you provided. Thank you 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
