Skip to content

Commit 93b7c2b

Browse files
committed
address review comments
1 parent 1908bae commit 93b7c2b

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

mlir/lib/Dialect/Arith/IR/InferIntRangeInterfaceImpls.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ using namespace mlir;
1919
using namespace mlir::arith;
2020
using namespace mlir::intrange;
2121

22-
intrange::OverflowFlags
22+
static intrange::OverflowFlags
2323
convertArithOverflowFlags(arith::IntegerOverflowFlags flags) {
2424
intrange::OverflowFlags retFlags = intrange::OverflowFlags::None;
2525
if (bitEnumContainsAny(flags, arith::IntegerOverflowFlags::nsw))

mlir/lib/Dialect/Index/IR/InferIntRangeInterfaceImpls.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ void BoolConstantOp::inferResultRanges(ArrayRef<ConstantIntRanges> argRanges,
4747
// Some arithmetic inference functions allow specifying special overflow / wrap
4848
// behavior. We do not require this for the IndexOps and use this helper to call
4949
// the inference function without any `OverflowFlags`.
50-
std::function<ConstantIntRanges(ArrayRef<ConstantIntRanges>)>
50+
static std::function<ConstantIntRanges(ArrayRef<ConstantIntRanges>)>
5151
inferWithoutOverflowFlags(InferRangeWithOvfFlagsFn inferWithOvfFn) {
5252
return [inferWithOvfFn](ArrayRef<ConstantIntRanges> argRanges) {
5353
return inferWithOvfFn(argRanges, OverflowFlags::None);

0 commit comments

Comments
 (0)