[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