Unverified Commit 5f0f269d authored by Aiden Grossman's avatar Aiden Grossman Committed by GitHub
Browse files

[MemRef] Fix -Wunused-function (#194366) 

areIndicesInBounds is only used within an assert statement, so mark it
[[maybe_unused]] so that the compiler does not otherwise warn in
non-assertion builds.
parent 9b2411da

mlir/lib/Dialect/MemRef/Transforms/ElideReinterpretCast.cpp

+1 −1
Original line number Diff line number Diff line
@@ -323,7 +323,7 @@ static bool isPureRankExpansionOrCollapsingRC(memref::ReinterpretCastOp rc) { 
/// Checks statically known and constant indices accessed by a load from a pure

/// rank expansion/collapsing to ensure in-bounds only access. Fully dynamic

/// indices are skipped (there is no way to verify them).

static bool areIndicesInBounds(memref::LoadOp load) {

[[maybe_unused]] static bool areIndicesInBounds(memref::LoadOp load) {

  auto rc = load.getMemRef().getDefiningOp<memref::ReinterpretCastOp>();

  auto rcOutputTy = cast<MemRefType>(rc.getResult().getType());