[PATCH] D52779: AMD BdVer2 (Piledriver) Initial Scheduler model

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: lib/Target/X86/X86.td:1025
 // Piledriver
-def : Proc<"bdver2", [
+def : ProcessorModel<"bdver2", BdVer2Model, [
   FeatureX87,
----------------
craig.topper wrote:
> Should we apply this to bdver3/4 as well? Are they similar?
Hmm. It would be ok to apply this to bdver1.

But i'm not sure about steamroller/excavator.
Those are more different, have loop buffer, 3 FPU pipes instead of 4, etc.


Repository:
  rL LLVM

https://reviews.llvm.org/D52779