takuto.ikuta added a comment.

Thank you for quick review!



================
Comment at: clang/lib/Sema/SemaDeclCXX.cpp:5715
+          (MD->isThisDeclarationADefinition() ||
+           MD->isImplicitlyInstantiable()) &&
+          TSK != TSK_ExplicitInstantiationDeclaration &&
----------------
hans wrote:
> The isThisDeclarationADefinition() and isImplicitlyInstantiable() checks 
> don't look right. Just checking isInlined() should be enough.
I see.


https://reviews.llvm.org/D51340



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

Reply via email to