[PATCH] D156538: [AArch64] Try to combine FMUL with FDIV

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 29 02:45:34 PDT 2023


dmgreen added a comment.

Oh I see, I hadn't spotted performFDivCombine. It was the scalar patterns I was thinking about, via IntegerToFP and fixedpoint_f32_i32 and SelectCVTFixedPosOperand.

Does it make sense to keep them selection fdiv, or should they always just match fmul? It would seem we only need one, and fmul is more canonical.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D156538/new/

https://reviews.llvm.org/D156538



More information about the llvm-commits mailing list