[PATCH] D109953: [X86] Combine the FADD(A, FMA(B, C, 0)) to FMA(B, C, A)

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 17 01:08:02 PDT 2021


xbolva00 added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:47481
 
 //  Try to combine the following nodes
 //  t21: v16f32 = X86ISD::VFMULC/VFCMULC t7, t8
----------------
Do we really need this output here? Simplify it a bit? Something like you wrote "Combine the FADD(A, FMA(B, C, 0)) to FMA(B, C, A)"?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109953



More information about the llvm-commits mailing list