[all-commits] [llvm/llvm-project] 26fedf: [MLIR] Generalize/complete getEnclosingAffineForIf...

Uday Bondhugula via All-commits all-commits at lists.llvm.org
Wed Aug 24 14:27:16 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 26fedf92c3122ff51fb27f434a8c5b5a47da8f94
      https://github.com/llvm/llvm-project/commit/26fedf92c3122ff51fb27f434a8c5b5a47da8f94
  Author: Uday Bondhugula <uday at polymagelabs.com>
  Date:   2022-08-25 (Thu, 25 Aug 2022)

  Changed paths:
    M mlir/include/mlir/Dialect/Affine/Analysis/Utils.h
    M mlir/lib/Dialect/Affine/Analysis/AffineAnalysis.cpp
    M mlir/lib/Dialect/Affine/Analysis/Utils.cpp

  Log Message:
  -----------
  [MLIR] Generalize/complete getEnclosingAffineForIfOps utility

Rename/generalize getEnclosingAffineForIfOps -> getEnclosingAffineOps.
The utility was originally written only for affine.for ops and then
extended for affine.if as well. It wasn't however updated when
affine.parallel was introduced -- in most cases, analysis has been used
in the presence of affine.for/if but not post parallelization. Extend
utility to also support affine.parallel ops; this allows future changes
to enable affine analysis and opts in the presence of affine.parallel
ops. Fix related stale comments.

This is NFC for all use cases in tree.

Change an associated assert to a utility failure.

Reviewed By: ftynse

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




More information about the All-commits mailing list