[all-commits] [llvm/llvm-project] d97bc3: Reapply "Extend getBackwardSlice to track values c...

Ian Wood via All-commits all-commits at lists.llvm.org
Fri Nov 1 08:42:33 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: d97bc388fd9ef8bc38353f93ff42d894ddc4a271
      https://github.com/llvm/llvm-project/commit/d97bc388fd9ef8bc38353f93ff42d894ddc4a271
  Author: Ian Wood <75152913+IanWood1 at users.noreply.github.com>
  Date:   2024-11-01 (Fri, 01 Nov 2024)

  Changed paths:
    M mlir/include/mlir/Analysis/SliceAnalysis.h
    M mlir/lib/Analysis/SliceAnalysis.cpp
    M mlir/test/IR/slice.mlir
    M mlir/test/lib/IR/TestSlicing.cpp

  Log Message:
  -----------
  Reapply "Extend getBackwardSlice to track values captured… (#114452)

This commit fixes the failure in the original PR when building with
shared libs. The problem is that `visitUsedValuesDefinedAbove` is
defined in `MLIRTransformUtils`, but that lib depends on this lib
(`MLIRAnalysis`). To fix, I dropped the use of
`visitUsedValuesDefinedAbove` and use `Region::walk` to traverse values
defined above.

Reapplies PR https://github.com/llvm/llvm-project/pull/113478
Reverts PR https://github.com/llvm/llvm-project/pull/114432

This reverts commit a9a8351.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list