[VecLib] Fix: Restore DebugFlag state in ReplaceWithVecLibTest

[paschalis-mpeis]

It appears that Google Tests run multiple modules from the same invocation.
As a result setting llvm::DebugFlag in this pass kept it on in subsequent passes, which is not the expected behavior.

[llvmbot]

@llvm/pr-subscribers-llvm-analysis

Author: Paschalis Mpeis (paschalis-mpeis)

Changes

It appears that Google Tests run multiple modules from the same invocation.
As a result setting llvm::DebugFlag in this pass kept it on in subsequent passes, which is not the expected behavior.

---

Full diff: https://github.com/llvm/llvm-project/pull/78989.diff

1 Files Affected:

• (modified) llvm/unittests/Analysis/ReplaceWithVecLibTest.cpp (+2)

diff --git a/llvm/unittests/Analysis/ReplaceWithVecLibTest.cpp b/llvm/unittests/Analysis/ReplaceWithVecLibTest.cpp
index a1f0a4a894c8d1..95f17f93bf4c0b 100644
--- a/llvm/unittests/Analysis/ReplaceWithVecLibTest.cpp
+++ b/llvm/unittests/Analysis/ReplaceWithVecLibTest.cpp
@@ -72,9 +72,11 @@ class ReplaceWithVecLibTest : public ::testing::Test {
     PB.registerFunctionAnalyses(FAM);
 
     // Enable debugging and capture std error
+    bool DebugFlagPrev = llvm::DebugFlag;
     llvm::DebugFlag = true;
     testing::internal::CaptureStderr();
     FPM.run(*M->getFunction("foo"), FAM);
+    llvm::DebugFlag = DebugFlagPrev;
     return getLastLine(testing::internal::GetCapturedStderr());
   }
 };

[mgabka]

LGTM, not sure if any extra changes added to the tests because of this bug needs to be reverted or not, but the change added in this PR looks correct to me.

[huntergr-arm]

LGTM