[PATCH] D157628: [AArch64][SVE2] Change the cost of extends with S/URHADD to 0

[David Sherwood via Phabricator via llvm-commits]

david-arm added a comment.

LGTM with nit addressed! Grazie @kmclaughlin. :)



================
Comment at: llvm/test/Analysis/CostModel/AArch64/ext-rhadd.ll:71
+
+define void @urhadd_i32_sext_i64_fixed(ptr %a, ptr %b, ptr %dst) {
+; SVE-LABEL: 'urhadd_i32_sext_i64_fixed'
----------------
nit: I think the test should be named `@urhadd_i32_zext_i64_fixed` and the same for other tests with the same issue below.


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

https://reviews.llvm.org/D157628