made isZeroDrop pass-by-reference

Author: mbagherbeikTT

mlir/include/mlir/Interfaces/SideEffectInterfaces.h

@@ -468,7 +468,7 @@ bool wouldOpBeTriviallyDead(Operation *op);
 /// Can only infer if loop is dead if it has constant loop bounds/steps.
 /// Otherwise we assume that it's dead to be conservative.
 ///
-std::optional<bool> isZeroTrip(mlir::LoopLikeOpInterface loop);
+std::optional<bool> isZeroTrip(mlir::LoopLikeOpInterface &loop);
 
 /// Returns true if the given operation is allowed to be moved under the
 /// memory effects interface.

mlir/lib/Interfaces/SideEffectInterfaces.cpp

@@ -319,7 +319,7 @@ bool mlir::wouldOpBeTriviallyDead(Operation *op) {
   return wouldOpBeTriviallyDeadImpl(op);
 }
 
-std::optional<bool> mlir::isZeroTrip(mlir::LoopLikeOpInterface loop) {
+std::optional<bool> mlir::isZeroTrip(mlir::LoopLikeOpInterface &loop) {
   auto lbs = loop.getLoopLowerBounds();
   auto ubs = loop.getLoopUpperBounds();
   auto steps = loop.getLoopSteps();