[PATCH] Prevent hoisting fmul from THEN/ELSE to IF if there is fmsub/fmadd opportunity
hfinkel at anl.gov
hfinkel at anl.gov
Tue Feb 3 16:37:05 PST 2015
Adjusting for my comment below, LGTM.
================
Comment at: include/llvm/Target/TargetLowering.h:27
----------------
You don't want enableAggressiveFMAFusion(VT) here. enableAggressiveFMAFusion(VT) affects whether we combine to form an FMA even when the fmul has more than one use, combine FMAs with other nodes to form multiple FMAs, whether we combine them through fpext nodes, etc. -- but it does not affect fusion legality (and, thus, whether a mul will be combined with a fsub/fadd to from an FMA).
http://reviews.llvm.org/D7299
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list