[llvm] [DependenceAnalysis] Extending SIV to handle fusable loops (PR #128782)

via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 21 04:38:03 PDT 2025


amehsan wrote:

> > Personally, I’d prefer to avoid adding new features to an already broken pass, but the code itself LGTM.
> 
> FWIW I agree, increasing the complexity before fixing the soundness problem makes it even more difficult to enable any pass depending on DA

I suggest this kind of comment is made when RFC is posted or in the beginning of code review. I think it is also  important that development is not a frustrating experience for the people invovled. 

But going forward, we are OK with focusing on correctness issues of DA. We are also going to help in that direction.

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


More information about the llvm-commits mailing list