[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