[PATCH] D71656: [DAGCombiner] Check term use before applying aggressive FSUB optimisation
    Jay Foad via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Dec 19 01:15:42 PST 2019
    
    
  
foad added a comment.
Looks like this case was missed in rL291642 <https://reviews.llvm.org/rL291642>. Is it worth checking the other cases too, e.g. the next one at line 11896? Can you add your test to `fadd-fma-fmul-combine.ll` instead of creating a new test file?
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71656/new/
https://reviews.llvm.org/D71656
    
    
More information about the llvm-commits
mailing list