[PATCH] D18376: [MachineScheduler] Add support for store clustering

[Jun Bum Lim via llvm-commits]

junbuml added a comment.

> In http://reviews.llvm.org/D18376#399964, @atrick wrote:

> This is fine with me.

> 

> If there was some reason I didn't do this it was specific to the microarchitecture and downstream optimizations at that time. As >long as we get good results from this change I have no objection.


I didn't see clear performance gains in score with this change in my spec2006 test. However, with this change I observed overall better llvm stats; please see my previous comments showing differences of llvm stats for spec2006.

I'm going to commit this if there is any objection by tomorrow.


http://reviews.llvm.org/D18376