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

Evandro Menezes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 20 08:33:42 PDT 2018


evandro added inline comments.


================
Comment at: lib/Target/AArch64/AArch64TargetTransformInfo.cpp:965
+    unsigned EltBits = LT.second.getScalarSizeInBits();
+    if (EltBits == 64)
+      return LT.first;
----------------
Please, comment on what you are trying to achieve here.


================
Comment at: lib/Target/AArch64/AArch64TargetTransformInfo.cpp:974
+    unsigned EltBits = LT.second.getScalarSizeInBits();
+    if (EltBits == 64)
+      return LT.first;
----------------
And here too.


Repository:
  rL LLVM

https://reviews.llvm.org/D48172





More information about the llvm-commits mailing list