xazax.hun added a comment.

In https://reviews.llvm.org/D42301#984713, @a.sidorin wrote:

> > I do not see a test for the following changes:
> > 
> > - ASTImporter: don't add templated declarations into DeclContext
>
> It's in ASTImporterTest. It checks that the templated decl cannot be found in 
> the enclosing TU.


Oh indeed, it looks likeI missed that, thanks :)

>> - ASTImporter: proper set ParmVarDecls for imported FunctionProtoTypeLoc
> 
> test/ASTMerge/function-cpp. If you remove the change, Sema will segfault.

Neat!


Repository:
  rC Clang

https://reviews.llvm.org/D42301



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

Reply via email to