[llvm] [SLP]Add runtime stride support for strided loads. (PR #81517)

Alexey Bataev via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 5 06:19:56 PST 2024


alexey-bataev wrote:

> LGTM
> 
> I'll comment that I find the guess and check style logic used in this patch to be very confusing without clear comments as to exactly what's going on. I think you could make this much clearer if you used appropriate variable names, or at least some clear comments about the point where you transfer from best effort guessing to checking your result and relying on it's correctness.

Renamed the variables, added some more comments

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


More information about the llvm-commits mailing list