[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