[PATCH] MachineCSE: Add a target query for the LookAheadLimit heurisitic

Matthias Braun matze at braunis.de
Mon May 4 10:33:00 PDT 2015


Hi Tom,

I don't know the history here, but as this does scan forward for each instruction of the basic block it looks like a way to avoid quadratic runtime behavior for (corner) cases with thousands of instructions in a basic block. I think it is no problem to go to a much higher limit than 5. But why go completely boundless, do you need a guarantee here that the CSE is happening?

- Matthias


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D9472

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list