rsmith accepted this revision. This revision is now accepted and ready to land.
================ Comment at: lib/Sema/SemaTemplateDeduction.cpp:1449-1451 @@ +1448,5 @@ + + const RecordType *RecordT = Arg->getAs<RecordType>(); + if (!RecordT) + return Result; + ---------------- Can you move this up and merge it with the `TDF_DerivedClass` check to also skip the copy in the case where the argument is not of class type? ================ Comment at: lib/Sema/SemaTemplateDeduction.cpp:1453-1457 @@ +1452,7 @@ + + // We cannot inspect base classes as part of deduction when the type + // is incomplete, so either instantiate any templates necessary to + // complete the type, or skip over it if it cannot be completed. + if (!S.isCompleteType(Info.getLocation(), Arg)) + return Result; + ---------------- ... This too, if it's not too awkward. http://reviews.llvm.org/D18868 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits