[cfe-commits] [PATCH] Adds a RecursiveASTVisitor test

Manuel Klimek klimek at google.com
Wed Apr 18 07:40:11 PDT 2012


Hi,

the attached patch adds a test for RAV based on the tooling
infrastructure. It also includes some FIXME tests which require fixes
in RAV itself / the AST and which are basically impossible to test
with an integration level test.

Some of the classes at the beginning in the file are probably going to
be pulled out when other tests want to use them (for example, in the
tooling branch I use the TestVisitor for tests of the refactoring
library). I have no idea where to pull them though, and wanted to get
feedback on the general idea first. My general feeling is that it's
still too much overhead to pull out a test like this, but I'd rather
have smaller steps here than overarchitect a solution.

Thoughts?

/Manuel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ravtest.patch
Type: application/octet-stream
Size: 8697 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20120418/99546989/attachment.obj>


More information about the cfe-commits mailing list