[llvm] [VPlan] Suppress leak in test (PR #114624)

via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 1 16:25:40 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-llvm-transforms

Author: Vitaly Buka (vitalybuka)

<details>
<summary>Changes</summary>

Probably needs a better fix. #<!-- -->114623


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


1 Files Affected:

- (modified) llvm/unittests/Transforms/Vectorize/VPlanTest.cpp (+3-2) 


``````````diff
diff --git a/llvm/unittests/Transforms/Vectorize/VPlanTest.cpp b/llvm/unittests/Transforms/Vectorize/VPlanTest.cpp
index 0f1b52bd453e0f..4a8615cc086b08 100644
--- a/llvm/unittests/Transforms/Vectorize/VPlanTest.cpp
+++ b/llvm/unittests/Transforms/Vectorize/VPlanTest.cpp
@@ -732,8 +732,9 @@ TEST(VPBasicBlockTest, print) {
 
   LLVMContext C;
   auto *ScalarHeader = BasicBlock::Create(C, "");
-  VPIRBasicBlock *ScalarHeaderVPBB = new VPIRBasicBlock(ScalarHeader);
-  VPlan Plan(VPBB0, TC, VPBB1, ScalarHeaderVPBB);
+  // FIXME: This looks wrong.
+  auto ScalarHeaderVPBB = std::make_unique<VPIRBasicBlock>(ScalarHeader);
+  VPlan Plan(VPBB0, TC, VPBB1, ScalarHeaderVPBB.get());
   std::string FullDump;
   raw_string_ostream OS(FullDump);
   Plan.printDOT(OS);

``````````

</details>


https://github.com/llvm/llvm-project/pull/114624


More information about the llvm-commits mailing list