[PATCH] D34504: [LLVM][X86][Goldmont] Adding new target-cpu: Goldmont

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 22 14:24:43 PDT 2017


RKSimon added a reviewer: RKSimon.
RKSimon added a comment.

Add goldmount to llvm\test\CodeGen\X86\cpus.ll

You might want to add it to atom-sched.ll and atom-call-reg-indirect.ll as well



================
Comment at: lib/Target/X86/X86Subtarget.h:517
   bool isSLM() const { return X86ProcFamily == IntelSLM; }
+  bool isGLM() const { return X86ProcFamily == IntelGLM; }
   bool useSoftFloat() const { return UseSoftFloat; }
----------------
It'd be better if this is wired into something so you can test it - lea fixup or tti cost models would be the most straightforward.

Although ideally X86ProcFamilyEnum would die and the atom-style cpus would be driven by features/cost models like other cpus.


https://reviews.llvm.org/D34504





More information about the llvm-commits mailing list