[PATCH] Allow TestVisitor-based tests to specify use of C++11

James Dennett jdennett at google.com
Fri Jun 28 13:19:03 PDT 2013


jdennett added you to the CC list for the revision "Allow TestVisitor-based tests to specify use of C++11".

Hi klimek, chandlerc,

Add enumerators to TestVisitor::Language to allow tests to explicitly specify use of C++98 or C++11.  Lang_CXX is preserved as an alias for Lang_CXX98.

This does not add Lang_CXX1Y or Lang_C11, on the assumption that it's better to add them if/when they are needed.

(This is a prerequisite for a test in a later patch for RecursiveASTVisitor.)

http://llvm-reviews.chandlerc.com/D1061

Files:
  unittests/Tooling/TestVisitor.h

Index: unittests/Tooling/TestVisitor.h
===================================================================
--- unittests/Tooling/TestVisitor.h
+++ unittests/Tooling/TestVisitor.h
@@ -39,14 +39,15 @@
 
   virtual ~TestVisitor() { }
 
-  enum Language { Lang_C, Lang_CXX };
+  enum Language { Lang_C, Lang_CXX98, Lang_CXX11, Lang_CXX=Lang_CXX98 };
 
   /// \brief Runs the current AST visitor over the given code.
   bool runOver(StringRef Code, Language L = Lang_CXX) {
     std::vector<std::string> Args;
     switch (L) {
       case Lang_C: Args.push_back("-std=c99"); break;
-      case Lang_CXX: Args.push_back("-std=c++98"); break;
+      case Lang_CXX98: Args.push_back("-std=c++98"); break;
+      case Lang_CXX11: Args.push_back("-std=c++11"); break;
     }
     return tooling::runToolOnCodeWithArgs(CreateTestAction(), Code, Args);
   }
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D1061.1.patch
Type: text/x-patch
Size: 847 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130628/1f4640a9/attachment.bin>


More information about the cfe-commits mailing list