martong marked 2 inline comments as done.
martong added inline comments.

================
Comment at: clang/unittests/AST/ASTImporterTest.cpp:5000
   EXPECT_TRUE(ImportedOK);
 }
 
----------------
balazske wrote:
> I see now that this test is really not required, the previous test checks the 
> almost same thing in case of `class B` and `NS` (but more test is always 
> better).
Actually, this test belongs to the previous patch , not to this one.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62375



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

Reply via email to