[PATCH] D38943: [ASTImporter] import SubStmt of CaseStmt

Rafael Stahl via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 16 05:51:06 PDT 2017


r.stahl added a comment.

Thanks for the fast response. See inline comment.



================
Comment at: unittests/AST/ASTImporterTest.cpp:100
 
+  // This might catch other cases.
+  Imported->dump(ToNothing);
----------------
xazax.hun wrote:
> I would elaborate a bit more on the purpose of the code below.
I will need help for that, since I do not have a better rationale myself. My guess would be that print has different assumptions and error checking than dump, so executing both increases coverage.


https://reviews.llvm.org/D38943





More information about the cfe-commits mailing list