[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