[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