[PATCH] D59485: [ASTImporter] Add an ImportInternal method to allow customizing Import behavior.

Balázs Kéri via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 22 08:27:04 PDT 2019


balazske added inline comments.


================
Comment at: clang/unittests/AST/ASTImporterTest.cpp:583
+  // Returns a ImporterConstructor that constructs this class.
+  static ASTImporterOptionSpecificTestBase::ImporterConstructor
+  getConstructor() {
----------------
balazske wrote:
> Is it possible to make this a static variable instead of function?
Another small thing: The comment above is now not updated ("Returns").


================
Comment at: clang/unittests/AST/ASTImporterTest.cpp:348
+
+    void setImporter(std::unique_ptr<ASTImporter> I) {
+      Importer = std::move(I);
----------------
This function is not needed?


================
Comment at: clang/unittests/AST/ASTImporterTest.cpp:482
 
+  ASTImporter &getImporter(Decl *From, Language ToLang) {
+    lazyInitToAST(ToLang, "", OutputFileName);
----------------
This is unused, can be removed.


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

https://reviews.llvm.org/D59485





More information about the cfe-commits mailing list