[PATCH] D52709: Add -instcombine-code-sinking option

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

> lib/Transforms/Scalar/Sink.cpp is probably a good starting point

I did an experiment with this recently.  It seems to be in a pretty good state already; it reduced codesize for an ARM codebase I was looking at by about by about 0.1%.  Haven't done any careful benchmarking, though.

Adding the command-line flag seems like enough to experiment. Not sure adding a flag to createInstructionCombiningPass() is really productive; it won't be useful once we turn off sinking by default.


https://reviews.llvm.org/D52709