[Mlir-commits] [mlir] [mlir] [dataflow] further optimize dataflow compile time (PR #149804)

[donald chen]

cxy-1993 wrote:

> You don't need to copy the entire analysis, as your analysis will be derived from the `DenseForwardDataFlowAnalysis` anyway you can do:
> 
> ```
> MyAnalysis::processOperation(Operation *op) {
>     if (something)
>         return success()
>         
>     return DenseForwardDataFlowAnalysis::processOperation(op);
> }
> ```

You're right, but this would also require maintaining an additional analysis. Furthermore, this scenario is common in dense analysis. On some downstream test cases, reducing these irrelevant analyses can decrease initialization time by a quarter.Therefore, I think it's reasonable to add this to DenseAnalysis.

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