[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