[PATCH] D21728: [AARCH64] Add Broadcom Vulcan scheduling model

[Renato Golin via llvm-commits]

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

Hi Pankaj,

I don't know much about the Vulcan model, so I'll trust you on the numbers. :)

But the code looks good to me with the extra comment removed.

cheers,
--renato


================
Comment at: lib/Target/AArch64/AArch64SchedVulcan.td:445
@@ +444,3 @@
+
+// Load vector reg, literal
+// Load vector reg, unscaled immed
----------------
I don't think you need this section here... you can easily say above that it's valid for FP, too.


http://reviews.llvm.org/D21728