[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