[Mlir-commits] [mlir] [mlir][dataflow] Fix for integer range analysis propagation bug (PR #93199)
Felix Schneider
llvmlistbot at llvm.org
Thu May 23 07:59:57 PDT 2024
ubfx wrote:
I agree that the control flow issue you have laid out is a problem. But to me it seems like assigning "fake" ranges to non-Integer values might lead to other problems down the line.
Maybe we should think about this check? Why don't we do `setAllToEntryStates()` on the Operation's results when any of the operands are unitialized?
https://github.com/llvm/llvm-project/blob/main/mlir/lib/Analysis/DataFlow/IntegerRangeAnalysis.cpp#L76
https://github.com/llvm/llvm-project/pull/93199
More information about the Mlir-commits
mailing list