[PATCH] D137913: [X86] Rewrite `getScalarizationOverhead()`

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: llvm/lib/Target/X86/X86TargetTransformInfo.cpp:4424
         const int CostValue = *LT.first.getValue();
         assert(CostValue >= 0 && "Negative cost!");
+        assert((SizeInBits % 128) == 0 && "Illegal vector");
----------------
This confused me - we might need a better term that CostValue now (NumLegalVectors?)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137913/new/

https://reviews.llvm.org/D137913