[Mlir-commits] [mlir] [mlir][dataflow] Drop the firstIndex argument of old visitNonControlFlowArguments, add new visitNonControlFlowArguments API and use it in IntegerRangeAnalysis (PR #175210)

[lonely eagle]

linuxlonelyeagle wrote:

> Overall, this makes sense to me, but can you split this up in two PRs: adding the second `visitNonControlFlowArgumentsImpl` overload in a separate PR.
> 
> This PR would then just drop the `firstIndex` parameter, change `successorInputs` -> `nonSuccessorInputs`, and pass the `argLattices` that correspond to the `nonSuccessorInputs`.
> 
> Doing this in two PRs should make it easier for folks to understand the changes and integrate.

"Thanks for the suggestion. I'll update it when I have some time.

https://github.com/llvm/llvm-project/pull/175210