[all-commits] [llvm/llvm-project] 995e9d: [MLIR] Fix getCommonBlock utility in affine analysis

Uday Bondhugula via All-commits all-commits at lists.llvm.org
Fri Jul 29 18:45:11 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 995e9d84f8f90dd237871d15cf7237866902e5b2
      https://github.com/llvm/llvm-project/commit/995e9d84f8f90dd237871d15cf7237866902e5b2
  Author: Uday Bondhugula <uday at polymagelabs.com>
  Date:   2022-07-30 (Sat, 30 Jul 2022)

  Changed paths:
    M mlir/lib/Dialect/Affine/Analysis/AffineAnalysis.cpp
    M mlir/test/Dialect/Affine/scalrep.mlir

  Log Message:
  -----------
  [MLIR] Fix getCommonBlock utility in affine analysis

Fix the hardcoded check for `FuncOp` in `getCommonBlock` utility: the
check should have been for an op that starts an affine scope. The
incorrect block returned in turn causes dependence analysis to function
incorrectly.

This change allows affine store-load forwarding to work correctly inside
any ops that start an affine scope.

Reviewed By: ftynse, dcaballe

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




More information about the All-commits mailing list