[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