[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