[CodeGen] LiveIntervalUnions::Array Implement move constructor

[optimisan]

Solves the double free error.

[optimisan]

• [NewPM][AMDGPU] Port SIPreAllocateWWMRegs to NPM #109939
• [AMDGPU] Add tests for SIPreAllocateWWMRegs #109963
• [NewPM][CodeGen] Port LiveRegMatrix to NPM #109938
• [CodeGen] LiveIntervalUnions::Array Implement move constructor #111357 👈
• [AMDGPU] Correct pass dependencies for SILowerSGPRSpills #109937
• [NewPM][CodeGen] Port VirtRegMap to NPM #109936
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @optimisan and the rest of your teammates on Graphite

[llvmbot]

@llvm/pr-subscribers-llvm-regalloc

Author: Akshat Oke (Akshat-Oke)

Changes

Solves the double free error.

---

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

1 Files Affected:

• (modified) llvm/include/llvm/CodeGen/LiveIntervalUnion.h (+7)

diff --git a/llvm/include/llvm/CodeGen/LiveIntervalUnion.h b/llvm/include/llvm/CodeGen/LiveIntervalUnion.h
index 81003455da4241..cc0f2a45bb182c 100644
--- a/llvm/include/llvm/CodeGen/LiveIntervalUnion.h
+++ b/llvm/include/llvm/CodeGen/LiveIntervalUnion.h
@@ -176,6 +176,13 @@ class LiveIntervalUnion {
     Array() = default;
     ~Array() { clear(); }
 
+    Array(Array &&Other) : Size(Other.Size), LIUs(Other.LIUs) {
+      Other.Size = 0;
+      Other.LIUs = nullptr;
+    }
+
+    Array(const Array &) = delete;
+
     // Initialize the array to have Size entries.
     // Reuse an existing allocation if the size matches.
     void init(LiveIntervalUnion::Allocator&, unsigned Size);

[optimisan]

Merge activity

• Oct 22, 5:41 AM EDT: A user started a stack merge that includes this pull request via Graphite.
• Oct 22, 5:51 AM EDT: Graphite rebased this pull request as part of a merge.
• Oct 22, 5:53 AM EDT: A user merged this pull request with Graphite.