[PATCH] Disable DAGCombine for -O0 and optnone
    Oleg Ranevskyy 
    llvm.mail.list at gmail.com
       
    Wed Jun  3 05:48:10 PDT 2015
    
    
  
In http://reviews.llvm.org/D9992#182775, @mkuper wrote:
> I'm not sure it's a good idea to move the entire combine into lowering.
>  The FNEG treatment is an actual combine. And I'm not 100% it will work correctly now (e.g. in cases where there are multiple users for the FNEG).
>
> It would probably be better to leave the combine as is, but add a trivial "safety-net" lowering for ISD::FMA -> X86ISD::FMADD.
Thanks Michael. I will look into this.
REPOSITORY
  rL LLVM
http://reviews.llvm.org/D9992
EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
    
    
More information about the llvm-commits
mailing list