diff options
| author | Mahesh Ravishankar <ravishankarm@google.com> | 2022-11-11 17:10:38 +0000 |
|---|---|---|
| committer | Mahesh Ravishankar <ravishankarm@google.com> | 2022-11-15 20:07:35 +0000 |
| commit | fc367dfa6770fda4adc4e5de79846f22e7e4e215 (patch) | |
| tree | 271b33935aaeaac87e22f05dcfccf9e0933454d0 /mlir/lib/Transforms/LoopInvariantCodeMotion.cpp | |
| parent | 9332ddfba69c38f9ceef4175b042fba0bb8e67bb (diff) | |
| download | llvm-fc367dfa6770fda4adc4e5de79846f22e7e4e215.zip llvm-fc367dfa6770fda4adc4e5de79846f22e7e4e215.tar.gz llvm-fc367dfa6770fda4adc4e5de79846f22e7e4e215.tar.bz2 | |
[mlir] Remove `Transforms/SideEffectUtils.h` and move the methods into `Interface/SideEffectInterfaces.h`.
The methods in `SideEffectUtils.h` (and their implementations in
`SideEffectUtils.cpp`) seem to have similar intent to methods already
existing in `SideEffectInterfaces.h`. Move the decleration (and
implementation) from `SideEffectUtils.h` (and `SideEffectUtils.cpp`)
into `SideEffectInterfaces.h` (and `SideEffectInterface.cpp`).
Also drop the `SideEffectInterface::hasNoEffect` method in favor of
`mlir::isMemoryEffectFree` which actually recurses into the operation
instead of just relying on the `hasRecursiveMemoryEffectTrait`
exclusively.
Differential Revision: https://reviews.llvm.org/D137857
Diffstat (limited to 'mlir/lib/Transforms/LoopInvariantCodeMotion.cpp')
| -rw-r--r-- | mlir/lib/Transforms/LoopInvariantCodeMotion.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mlir/lib/Transforms/LoopInvariantCodeMotion.cpp b/mlir/lib/Transforms/LoopInvariantCodeMotion.cpp index 4711937..854fde0 100644 --- a/mlir/lib/Transforms/LoopInvariantCodeMotion.cpp +++ b/mlir/lib/Transforms/LoopInvariantCodeMotion.cpp @@ -13,8 +13,8 @@ #include "mlir/Transforms/Passes.h" #include "mlir/Interfaces/LoopLikeInterface.h" +#include "mlir/Interfaces/SideEffectInterfaces.h" #include "mlir/Transforms/LoopInvariantCodeMotionUtils.h" -#include "mlir/Transforms/SideEffectUtils.h" namespace mlir { #define GEN_PASS_DEF_LOOPINVARIANTCODEMOTION |