[PATCH] D156538: [AArch64] Try to combine FMUL with FDIV
Sam Tebbs via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 1 08:45:23 PDT 2023
samtebbs added a comment.
I think this is good as it is, although I'm not 100% sure on the fact that we need to get it converted to `aarch64_neon_vcvtfxs2fp` first, as if something goes wrong there then we'll miss out on this optimisation. If you can think of a way to circumvent the need for that and go directly to the scvtf that would be good, otherwise this looks good to me.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156538/new/
https://reviews.llvm.org/D156538
More information about the llvm-commits
mailing list