[llvm] [NFC][LoopVectorize] Cache result of requiresScalarEpilogue (PR #108981)
David Sherwood via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 18 07:07:05 PDT 2024
david-arm wrote:
> Would it be simpler to have a separate helper `setRequiresScalarEpilogue` that's called once up-front with `requiresScalarEpilogue` simply returning the decision as done in multiple other places?
OK, I've tried doing this, but I realised there are places where we have to invalidate the decision due to changes in the scalar epilogue status or interleave groups. However, that also means my first version was also incorrect even though all tests passed!
https://github.com/llvm/llvm-project/pull/108981
More information about the llvm-commits
mailing list