[all-commits] [llvm/llvm-project] ac38a7: [MLIR] Fix incorrect assertion in memref replaceme...

Uday Bondhugula via All-commits all-commits at lists.llvm.org
Fri Nov 18 23:05:57 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ac38a7d3524733bee3901d2d8ea20ad664bbb7fc
      https://github.com/llvm/llvm-project/commit/ac38a7d3524733bee3901d2d8ea20ad664bbb7fc
  Author: Uday Bondhugula <uday at polymagelabs.com>
  Date:   2022-11-19 (Sat, 19 Nov 2022)

  Changed paths:
    M mlir/lib/Dialect/Affine/Utils/Utils.cpp

  Log Message:
  -----------
  [MLIR] Fix incorrect assertion in memref replacement utility

The assertion had an unchecked use of getDefiningOp. The condition
itself is stale and not needed. This is a trivial/obvious fix - the
buggy path wasn't exercised since the current passes/users of this
utility always passed the result of an op as an "extra index".

Differential Revision: https://reviews.llvm.org/D138191




More information about the All-commits mailing list