[PATCH] D66088: AMD Znver2 (Rome) Scheduler enablement

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 13 14:11:12 PST 2019


craig.topper added a comment.

In D66088#1740344 <https://reviews.llvm.org/D66088#1740344>, @lebedev.ri wrote:

> Not sure how this (any) sched model should be reviewed.
>  Perhaps it would help to post the exegesis reports.
>
> FWIW i'd just merge this.


I agree with Roman that there's not a good way to review this other than comparing with exegesis data. Sounds like from the initial description there is planned follow up with more data from exegesis. So I think this is fine to start with.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66088/new/

https://reviews.llvm.org/D66088





More information about the llvm-commits mailing list