[PATCH] D95598: [AArch64][SVE]Add cost model for broadcast shuffle

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 1 01:12:56 PST 2021


david-arm accepted this revision.
david-arm added a comment.
This revision is now accepted and ready to land.

LGTM with nit addressed!



================
Comment at: llvm/test/Analysis/CostModel/AArch64/sve-shuffle-broadcast.ll:11
+
+define void  @broadcast(<vscale x 32 x i8> %v) #0{
+; CHECK-LABEL: 'broadcast':
----------------
nit: It looks like '%v' isn't used anymore here so can delete?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95598



More information about the llvm-commits mailing list