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

[Hal Finkel via Phabricator via llvm-commits]

hfinkel added a comment.

I think that this is fine, as a work around, but I don't quite understand the problem based on this description. I'm thinking that either:

1. Like on the https://reviews.llvm.org/P8 (and previous), there are some in-order resources in the decoder/dispatcher (which should be modeled as in-order resources).
2. There are resources for which we're significantly underestimating the buffer size (the current https://reviews.llvm.org/P9 scheduling model only sets the BufferSize for BR (which is fine, but as we don't currently schedule branches, it's essentially documentation). Maybe we should be setting BufferSize for other things.

(or some combination of the two). Based on the description in the patch and in the patch description, it sounds like the latter, but I'm not sure.


https://reviews.llvm.org/D39805