[PATCH] D39805: [Power9] Set MicroOpBufferSize for Power 9

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 6 15:24:00 PST 2017


echristo added a comment.

In https://reviews.llvm.org/D39805#946984, @stefanp wrote:

> Hi Everyone, 
>  Thank you for your comments.
>
> In light of the feedback I'm thinking of changing this patch a little. We can set the MicroOpBufferSize = 44 since that's the actual size of the buffer.


Feel free to add this part immediately :)

IMO "represent the actual hardware faithfully" is, as much as can be, a "non-controversial" change.

> To get the scheduler behavior we want we can set some of the resources to BufferSize=1 so that CurrCycle will increase as we expect.

This should probably get reviewed.

Thanks!

-eric


https://reviews.llvm.org/D39805





More information about the llvm-commits mailing list