rnk added inline comments.

================
Comment at: lib/Sema/SemaDeclCXX.cpp:4813
@@ +4812,3 @@
+        // and move constructor, so don't attempt to import/export them if
+        // we have a definition.
+        auto *CXXC = dyn_cast<CXXConstructorDecl>(MD);
----------------
Oh, so we were already doing this check. I don't see what's wrong with our 
current behavior, though. We export a few more symbols than MSVC 2013, but 
there's no ABI problem with that.

http://reviews.llvm.org/D19156



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

Reply via email to