-
Notifications
You must be signed in to change notification settings - Fork 13.6k
Remove a debugging message left in a unit test. #137451
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@llvm/pr-subscribers-llvm-transforms Author: Abid Qadeer (abidh) ChangesThis should fix the regression reported in #136016 (comment) Full diff: https://github.com/llvm/llvm-project/pull/137451.diff 1 Files Affected:
diff --git a/llvm/unittests/Transforms/Utils/CodeExtractorTest.cpp b/llvm/unittests/Transforms/Utils/CodeExtractorTest.cpp
index 42a00c9beb560..239e84d6024ff 100644
--- a/llvm/unittests/Transforms/Utils/CodeExtractorTest.cpp
+++ b/llvm/unittests/Transforms/Utils/CodeExtractorTest.cpp
@@ -787,7 +787,6 @@ TEST(CodeExtractor, ArgsDebugInfo) {
CE.findAllocas(CEAC, SinkingCands, HoistingCands, CommonExit);
CE.findInputsOutputs(Inputs, Outputs, SinkingCands);
Function *Outlined = CE.extractCodeRegion(CEAC, Inputs, Outputs);
- Outlined->dump();
EXPECT_TRUE(Outlined);
BasicBlock &EB = Outlined->getEntryBlock();
Instruction *Term = EB.getTerminator();
|
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/190/builds/19044 Here is the relevant piece of the build log for the reference
|
This PR just removes a debugging message from a unit test. There is not any functionality change. I am not sure how this can cause any other test failure. |
This should fix the regression reported in llvm#136016 (comment)
This should fix the regression reported in llvm#136016 (comment)
This should fix the regression reported in llvm#136016 (comment)
This should fix the regression reported in llvm#136016 (comment)
This should fix the regression reported in llvm#136016 (comment)
This should fix the regression reported in #136016 (comment)