Author: filcab Date: Mon Mar 2 12:49:37 2015 New Revision: 230984 URL: http://llvm.org/viewvc/llvm-project?rev=230984&view=rev Log: Add -frtti and -fexceptions to tests that assume these are on.
Summary: We now have targets that don't enable rtti/exceptions by default, and the ASTMatchers tests are assuming that these features are on (e.g: They use dynamic_cast or try). Reviewers: klimek, thakis, djasper Subscribers: klimek, cfe-commits Differential Revision: http://reviews.llvm.org/D7892 Modified: cfe/trunk/unittests/ASTMatchers/ASTMatchersTest.h Modified: cfe/trunk/unittests/ASTMatchers/ASTMatchersTest.h URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/unittests/ASTMatchers/ASTMatchersTest.h?rev=230984&r1=230983&r2=230984&view=diff ============================================================================== --- cfe/trunk/unittests/ASTMatchers/ASTMatchersTest.h (original) +++ cfe/trunk/unittests/ASTMatchers/ASTMatchersTest.h Mon Mar 2 12:49:37 2015 @@ -73,7 +73,11 @@ testing::AssertionResult matchesConditio std::unique_ptr<FrontendActionFactory> Factory( newFrontendActionFactory(&Finder)); // Some tests use typeof, which is a gnu extension. - std::vector<std::string> Args(1, CompileArg); + std::vector<std::string> Args; + Args.push_back(CompileArg); + // Some tests need rtti/exceptions on + Args.push_back("-frtti"); + Args.push_back("-fexceptions"); if (!runToolOnCodeWithArgs(Factory->create(), Code, Args, "input.cc", VirtualMappedFiles)) { return testing::AssertionFailure() << "Parsing error in \"" << Code << "\""; _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
