This is an archive of the discontinued LLVM Phabricator instance.

Differential D45188

Remove unused local variable in CFIVerifyTests test fixture
AbandonedPublic

Authored by yursha on Apr 2 2018, 2:51 PM.

Details

Reviewers
vlad.tsyrklevich
Summary

Remove unused local variable in CFIVerifyTests test fixture.

Diff Detail

Event Timeline

yursha created this revision.Apr 2 2018, 2:51 PM
Herald added a subscriber: a.sidorin. · View Herald TranscriptApr 2 2018, 2:52 PM
Harbormaster completed remote builds in B16643: Diff 140694.Apr 2 2018, 2:53 PM
yursha abandoned this revision.Apr 2 2018, 4:25 PM

Please, disregard

Revision Contents

PathSize
unittests/
tools/
llvm-cfi-verify/
1 line

Diff 140694

unittests/tools/llvm-cfi-verify/GraphBuilder.cpp

Show First 20 Lines • Show All 120 Lines • ▼ Show 20 Linespublic:
Error initialiseDisassemblyMembers() { Error initialiseDisassemblyMembers() {
return FileAnalysis::initialiseDisassemblyMembers(); return FileAnalysis::initialiseDisassemblyMembers();
} }
}; };
class BasicGraphBuilderTest : public ::testing::Test { class BasicGraphBuilderTest : public ::testing::Test {
protected: protected:
virtual void SetUp() { virtual void SetUp() {
IgnoreDWARFFlag = true;
SuccessfullyInitialised = true; SuccessfullyInitialised = true;
if (auto Err = Analysis.initialiseDisassemblyMembers()) { if (auto Err = Analysis.initialiseDisassemblyMembers()) {
handleAllErrors(std::move(Err), [&](const UnsupportedDisassembly &E) { handleAllErrors(std::move(Err), [&](const UnsupportedDisassembly &E) {
SuccessfullyInitialised = false; SuccessfullyInitialised = false;
outs() outs()
<< "Note: CFIVerifyTests are disabled due to lack of x86 support " << "Note: CFIVerifyTests are disabled due to lack of x86 support "
"on this build.\n"; "on this build.\n";
}); });
▲ Show 20 Lines • Show All 449 Lines • Show Last 20 Lines