[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
    
        - Previous message: [PATCH] D109953: [X86] Combine the FADD(A, FMA(B, C, 0)) to FMA(B, C,  A)
- Next message: [PATCH] D109953: [X86] Combine the FADD(A, FMA(B, C, 0)) to FMA(B, C,  A)
-  Messages sorted by: 
              [ date ]
              [ thread ]
              [ subject ]
              [ author ]
         
  
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
    
    
        
	- Previous message: [PATCH] D109953: [X86] Combine the FADD(A, FMA(B, C, 0)) to FMA(B, C,  A)
- Next message: [PATCH] D109953: [X86] Combine the FADD(A, FMA(B, C, 0)) to FMA(B, C,  A)
-  Messages sorted by: 
              [ date ]
              [ thread ]
              [ subject ]
              [ author ]
         
More information about the llvm-commits
mailing list