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

[Guozhi Wei via Phabricator via llvm-commits]

Carrot 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. 
>  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. 
>  The solution will still be a workaround but it will provide the correct result for isOutOfOrder() and it won't take as long to implement as it would take to write a custom PPC scheduler. Unfortunately, there is no plan yet of when we could start on a PPC specific scheduler.


This method will not impact other optimizations, so it sounds good to me.


https://reviews.llvm.org/D39805