[PATCH] D35319: LSE Atomics reorg - Part I

Chad Rosier via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 13 07:37:04 PDT 2017


mcrosier added a comment.

In https://reviews.llvm.org/D35319#807979, @steleman wrote:

> In https://reviews.llvm.org/D35319#807963, @t.p.northover wrote:
>
> > This diff covers lots of different areas:
>
>
>
>
> > 3. The scheduling: whatever, assuming you have inside knowledge of ThunderX.
>
> What does //whatever// mean, in this context?


The scheduling changes for ThunderX can't really be reviewed by the community (as we don't have the domain specific knowledge), so Tim is basically saying do whatever you see fit.


Repository:
  rL LLVM

https://reviews.llvm.org/D35319





More information about the llvm-commits mailing list