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

Stefan Pintilie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 6 11:40:07 PST 2017


stefanp added a comment.

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.


https://reviews.llvm.org/D39805





More information about the llvm-commits mailing list