[PATCH] D44644: [X86] Use silvermont cost model overrides for goldmont as well.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 25 02:55:10 PDT 2018


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D44644#1047765, @craig.topper wrote:

> Should I make a copy of the SLM scheduler and use it for GLM so we can start refining it?


Probably - I was looking to find tidy ways to override models such as these - architecturally the same but with a few latency tweaks - but couldn't see anything. Probably easier just to copy it,, maybe once you're happy with the accuracy of the SLM model.

These changes LGTM though


Repository:
  rL LLVM

https://reviews.llvm.org/D44644





More information about the llvm-commits mailing list