[llvm] [AArch64][LV][SLP] Vectorizers use getFRemInstrCost for frem costs (PR #82488)
Alexey Bataev via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 28 09:27:46 PST 2024
alexey-bataev wrote:
> True, but the newly introduced function makes it quite explicit to it's users.
> They can't use it by mistake without noticing. And now we'll have it in two places that we know are OK.
Same with adding the new parameter to the existing function, no difference at all.
> Exactly. Until codegen can fully handle frem, we'd prefer not to set any 'traps' for future patches and authors, and instead be very explicit about it.
As I said, this is not a problem of this patch
https://github.com/llvm/llvm-project/pull/82488
More information about the llvm-commits
mailing list