Skip to content

Commit 3e579d9

Browse files
committed
[VPlan] Fix unit test without LLVM_ENABLE_DUMP. NFC
Without dumping the faulty recipe isn't printed, so account for that like in the other tests. Fixes the buildbot failure at https://lab.llvm.org/buildbot/#/builders/2/builds/30229
1 parent 0c3092b commit 3e579d9

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

llvm/unittests/Transforms/Vectorize/VPlanVerifierTest.cpp

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -312,9 +312,14 @@ TEST_F(VPVerifierTest, NonHeaderPHIInHeader) {
312312
#endif
313313
EXPECT_FALSE(verifyVPlanIsValid(Plan));
314314
#if GTEST_HAS_STREAM_REDIRECTION
315+
#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
315316
EXPECT_STREQ(
316317
"Found non-header PHI recipe in header VPBB: IR <badref> = phi i32 \n",
317318
::testing::internal::GetCapturedStderr().c_str());
319+
#else
320+
EXPECT_STREQ("Found non-header PHI recipe in header VPBB",
321+
::testing::internal::GetCapturedStderr().c_str());
322+
#endif
318323
#endif
319324
}
320325

0 commit comments

Comments
 (0)