[PATCH] D39054: [X86][Broadwell] Added the instruction scheduling information for the Broadwell CPU.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 23 05:38:10 PDT 2017


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

LGTM.



================
Comment at: lib/Target/X86/X86.td:579
 ]>;
-class BroadwellProc<string Name> : ProcModel<Name, HaswellModel,
+class BroadwellProc<string Name> : ProcModel<Name, BroadwellModel,
                                              BDWFeatures.Value, [
----------------
gadi.haber wrote:
> RKSimon wrote:
> > Other CPUs are still using HaswellModel (KNL, KNM + CannonLake) - is that correct?
> Sorry for the delayed response. Was on a short vacation,
> The answer is yes: KNL, KNM and CannonLake are currently all using the Haswell Model.
OK none of this should affect this patch. I expected CannonLake to use the SkylakeServer model. The KNL/KNM architectures are very different to Haswell so I'd have expected something closer to SLM?


Repository:
  rL LLVM

https://reviews.llvm.org/D39054





More information about the llvm-commits mailing list