[PATCH] D103952: [CostModel][AArch64] Improve the cost estimate of CTPOP intrinsic

Caroline via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 10 00:26:39 PDT 2021


CarolineConcatto added inline comments.


================
Comment at: llvm/test/Analysis/CostModel/AArch64/ctpop.ll:64
 define <2 x i32> @test_ctpop_v2i32(<2 x i32> %a) {
-;
 ; CHECK-LABEL: 'test_ctpop_v2i32'
----------------
nit: I don't know why ';' was added here in all functions, but I don't think you should use this patch to remove them.


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

https://reviews.llvm.org/D103952



More information about the llvm-commits mailing list