[PATCH] D46276: [CostModel][X86] Derive TTI costs from complete scheduling models (PR36550) (RFC)

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 30 11:06:48 PDT 2018


craig.topper added inline comments.


================
Comment at: lib/Target/X86/X86TargetTransformInfo.cpp:658
+  static const SchedCostTblEntry AVX1CostTable[] = {
+    { ISD::FADD,  MVT::v4f64,   2, { X86::VADDPDYrr } },
+    { ISD::FADD,  MVT::v8f32,   2, { X86::VADDPSYrr } },
----------------
Is this going to cause a bunch of SmallVectors to be constructed at startup? I think that goes against our coding standards.


Repository:
  rL LLVM

https://reviews.llvm.org/D46276





More information about the llvm-commits mailing list