[Mlir-commits] [mlir] [mlir][ArmSME] Suppress potential unused warning (PR #99573)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Thu Jul 18 14:39:41 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir

Author: Chenguang Wang (wecing)

<details>
<summary>Changes</summary>

When building in release mode, the assert will be dropped, making `remove` unused.

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


1 Files Affected:

- (modified) mlir/lib/Dialect/ArmSME/Transforms/TileAllocation.cpp (+1) 


``````````diff
diff --git a/mlir/lib/Dialect/ArmSME/Transforms/TileAllocation.cpp b/mlir/lib/Dialect/ArmSME/Transforms/TileAllocation.cpp
index 5cac770b03ed1..4648944156951 100644
--- a/mlir/lib/Dialect/ArmSME/Transforms/TileAllocation.cpp
+++ b/mlir/lib/Dialect/ArmSME/Transforms/TileAllocation.cpp
@@ -619,6 +619,7 @@ void allocateTilesToLiveRanges(
         // Remove the live range from the active/inactive sets.
         if (!activeRanges.remove(rangeToSpill)) {
           bool removed = inactiveRanges.remove(rangeToSpill);
+          (void)removed;
           assert(removed && "expected a range to be removed!");
         }
       }

``````````

</details>


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


More information about the Mlir-commits mailing list