rnk added inline comments. ================ Comment at: lib/Sema/SemaType.cpp:5876 @@ +5875,3 @@ + + if (!IsCtorOrDtor) { + if (CurCC != DefaultCC || DefaultCC == ToCC) ---------------- andreybokhanko wrote: > rnk wrote: > > This looks like the !IsCtorOrDtor check that affects Itanium. Isn't it > > already handled for MS C++ above? > Reid, now I'm completely confused... I thought you asked me to insert this > check for Itanium ABI as well (in your comment from Sep 1st). Am I mistaken? > If yes, I can remove it and restore ctor-dtor-alias.cpp test back to original > version. Sorry, it shouldn't affect Itanium. What you have without this check is fine.
http://reviews.llvm.org/D12402 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits