[PATCH] D14286: ASTImporter: expressions, pt.1

Serge Pavlov via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 25 09:38:47 PST 2015


sepavloff added a comment.

If you are going to make unit tests, consider alternative approach also. In repository `test-suite` the directory `SingleSource/UnitTests` contains runtime tests obtained from single file. You could modify Makefile that runs these tests in such a way that each test runs twice, one as now it runs and the other would run through the sequence `compile to ast` - `compile empty source with with ast-import`. Tests for missing nodes could be added into `SingleSource/UnitTests`, they would be substantially simpler that those using ASTMatchers. More importantly, such tests would make indeed exhaustive testing of importer.


http://reviews.llvm.org/D14286





More information about the cfe-commits mailing list