[PATCH] Disable DAGCombine for -O0 and optnone

Michael Kuperstein michael.m.kuperstein at intel.com
Wed Jun 3 01:03:45 PDT 2015


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.


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