squiid/llvm
1
0
Fork 0
Code Issues Pull requests Activity

[AST][NFC] Simplify a regression test

Browse source 
Differential Revision: https://reviews.llvm.org/D83438
This commit is contained in:
Jan Korous 2020-07-16 12:06:02 -07:00
parent 29f8c9f6c2
commit 5e8b4be9f8
3 changed files with 6 additions and 88 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
clang/test/AST/regression-new-expr-crash.cpp
View file

@ -1,4 +1,4 @@
// RUN: %clang_cc1 -triple x86_64-unknown-unknown -fsyntax-only %s
// RUN: %clang_cc1 -triple x86_64-unknown-unknown -fsyntax-only -verify %s
struct Bar {int a;};
const Bar arr[2] = {{1}};
@ -11,3 +11,8 @@ void foo(int a) {
Foo *foo_array;
foo_array = new Foo[arr[0].a];
}
void Test(int N) {
int arr[N];
decltype([&arr]{}) *p; // expected-error {{lambda expression in an unevaluated operand}}
}

1
clang/unittests/AST/CMakeLists.txt
View file

@ -26,7 +26,6 @@ add_clang_unittest(ASTTests
DeclTest.cpp
EvaluateAsRValueTest.cpp
ExternalASTSourceTest.cpp
HasSideEffectsTest.cpp
NamedDeclPrinterTest.cpp
RecursiveASTVisitorTest.cpp
SizelessTypesTest.cpp

86
clang/unittests/AST/HasSideEffectsTest.cpp
View file

@ -1,86 +0,0 @@
//===- unittest/AST/HasSideEffectsTest.cpp --------------------------------===//
//
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
// See https://llvm.org/LICENSE.txt for license information.
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
#include "clang/AST/RecursiveASTVisitor.h"
#include "clang/AST/ASTConsumer.h"
#include "clang/AST/ASTContext.h"
#include "clang/AST/Attr.h"
#include "clang/Frontend/FrontendAction.h"
#include "clang/Tooling/Tooling.h"
#include "llvm/ADT/FunctionExtras.h"
#include "llvm/ADT/STLExtras.h"
#include "gmock/gmock.h"
#include "gtest/gtest.h"
#include <cassert>
using namespace clang;
namespace {
class ProcessASTAction : public clang::ASTFrontendAction {
public:
ProcessASTAction(llvm::unique_function<void(clang::ASTContext &)> Process)
: Process(std::move(Process)) {
assert(this->Process);
}
std::unique_ptr<ASTConsumer> CreateASTConsumer(CompilerInstance &CI,
StringRef InFile) {
class Consumer : public ASTConsumer {
public:
Consumer(llvm::function_ref<void(ASTContext &CTx)> Process)
: Process(Process) {}
void HandleTranslationUnit(ASTContext &Ctx) override { Process(Ctx); }
private:
llvm::function_ref<void(ASTContext &CTx)> Process;
};
return std::make_unique<Consumer>(Process);
}
private:
llvm::unique_function<void(clang::ASTContext &)> Process;
};
class RunHasSideEffects
: public RecursiveASTVisitor<RunHasSideEffects> {
public:
RunHasSideEffects(ASTContext& Ctx)
: Ctx(Ctx) {}
bool VisitLambdaExpr(LambdaExpr *LE) {
LE->HasSideEffects(Ctx);
return true;
}
ASTContext& Ctx;
};
} // namespace
TEST(HasSideEffectsTest, All) {
llvm::StringRef Code = R"cpp(
void Test() {
int msize = 4;
float arr[msize];
[&arr] {};
}
)cpp";
ASSERT_NO_FATAL_FAILURE(
clang::tooling::runToolOnCode(
std::make_unique<ProcessASTAction>(
[&](clang::ASTContext &Ctx) {
RunHasSideEffects Visitor(Ctx);
Visitor.TraverseAST(Ctx);
}
),
Code)
);
}