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

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 13 11:53:57 PDT 2016


mcrosier 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.


Thanks, Andy.

I think the last bit that is needed is for @arsenm or @tstellarAMD to approve the patch as this may impact the AMD target.


http://reviews.llvm.org/D18376





More information about the llvm-commits mailing list