shafik accepted this revision.
shafik added a comment.
Besides my comments about different scenarios this LGTM
================
Comment at: clang/lib/AST/ASTImporter.cpp:3018
+
+bool ASTNodeImporter::hasAutoReturnTypeDeclaredInside(FunctionDecl *D) {
+ QualType FromTy = D->getType();
----------------
I added a comment with several examples of C++ code that I am not sure if this
will catch or not. Since C++17 we allow types to be declared in several more
places.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70819/new/
https://reviews.llvm.org/D70819
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
