[PATCH] MachineCSE: Add a target query for the LookAheadLimit heurisitic
hfinkel at anl.gov
hfinkel at anl.gov
Tue May 5 14:59:27 PDT 2015
REPOSITORY
rL LLVM
================
Comment at: include/llvm/Target/TargetInstrInfo.h:1242
@@ +1241,3 @@
+ // 5 is the value for this heuristic that used to be hard-coded into
+ // the MachineCSE pass. I have no idea why 5 was choosen.
+ return 5;
----------------
Can we just say something like, "The default lookahead is small to prevent unprofitable quadratic behavior." Next year, no one will know to whom "I" refers without a lot of work ;)
http://reviews.llvm.org/D9472
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list