[PATCH] D60461: [ASTImporter] Import TemplateParameterLists in function templates.

Aleksei Sidorin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Jul 7 15:31:24 PDT 2019


a_sidorin added a comment.

Post-LGTM :)



================
Comment at: lib/AST/ASTImporter.cpp:2789
+  if (Num == 0)
+    return Error::success();
+  SmallVector<TemplateParameterList *, 2> ToTPLists(Num);
----------------
Please add a newline after return.


================
Comment at: lib/AST/ASTImporter.cpp:2796
+    else
+      return ToTPListOrErr.takeError();
+  ToD->setTemplateParameterListsInfo(Importer.ToContext, ToTPLists);
----------------
Let'a add a newline here too.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60461/new/

https://reviews.llvm.org/D60461





More information about the cfe-commits mailing list