[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