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

[Gadi Haber via Phabricator via llvm-commits]

gadi.haber added inline comments.


================
Comment at: lib/Target/X86/X86.td:579
 ]>;
-class BroadwellProc<string Name> : ProcModel<Name, HaswellModel,
+class BroadwellProc<string Name> : ProcModel<Name, BroadwellModel,
                                              BDWFeatures.Value, [
----------------
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.


Repository:
  rL LLVM

https://reviews.llvm.org/D39054