[PATCH] D31827: Add a flag that enables more aggressive load PRE

[Daniel Berlin via Phabricator via llvm-commits]

dberlin added a comment.

I would strongly prefer to either turn this on by default (IE no option) or not do it.
Which means we need numbers.

It's not a useful knob to tweak, IMHO, and definitely not one amenable to good heuristics (the age old register allocation vs PRE debate).
For code size, PRE is generally a lose compared to VBE based hoisting.


https://reviews.llvm.org/D31827