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

[Guozhi Wei via Phabricator via llvm-commits]

Carrot added a comment.

It looks not a good work around to me. Because the problem is only in scheduler, but the patch inappropriately changes a very basic API isOutOfOrder(), it potentially can be used by many other optimizations. I personally have a patch depends on this API https://reviews.llvm.org/D39352.

A few more questions:

1. Can you find another work around local to the scheduler?

2. You mentioned an ideal solution is a long term https://reviews.llvm.org/P9 specific scheduler, do you have specific plan for it?

3. In the source code, you mentioned

// If, in the future, we decide to implement a PPC scheduler ...

  Can I understand it as in the future, you may also not implement a PPC scheduler?


https://reviews.llvm.org/D39805