[PATCH] D48172: [CostModel][AArch64] Add some initial costs for SK_Select and SK_PermuteSingleSrc

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: lib/Target/AArch64/AArch64TargetTransformInfo.cpp:964
+    static const CostTblEntry SelectTbl[] = {
+        {ISD::VECTOR_SHUFFLE, MVT::v2i32, 1},
+        {ISD::VECTOR_SHUFFLE, MVT::v4i32, 2},
----------------
evandro wrote:
> Independently of the costs, this lookup table could be simplified to a test on the number of vector elements.
That's trivial to do - the tables look easier to grok, but I'm happy to make the change.


Repository:
  rL LLVM

https://reviews.llvm.org/D48172