[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