[llvm] [DependenceAnalysis] Fix SIV test crash when no AddRec after propagation (PR #154980)

Ryotaro Kasuga via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 24 15:36:00 PDT 2025


https://github.com/kasuga-fj requested changes to this pull request.

> > Do not delete the assertion. The assertion error is correct. The right approach is to stop passing non-AddRecExpr values to testSIV.
> 
> I'm missing the context here. Are you speaking about this code:
> 
> ```
>   llvm_unreachable("SIV test expected at least one AddRec");
>   return false;
> ```

Yes. And as I wrote in another reply, the propagation is not relevant here. Again, the correct approach is to stop the analysis at an earlier stage.

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


More information about the llvm-commits mailing list