[PATCH] D55480: [CostModel][X86][AArch64] Adjust cost of the scalarization part of min/max reduction.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 9 03:57:24 PST 2018


RKSimon added inline comments.


================
Comment at: test/Transforms/SLPVectorizer/X86/horizontal-minmax.ll:1108
+; SSE-NEXT:    [[TMP26:%.*]] = select i1 [[TMP19]], i32 [[TMP17]], i32 [[TMP18]]
+; SSE-NEXT:    ret i32 [[OP_EXTRA]]
 ;
----------------
This codegen looks like the final icmp+selects are being done with the scalars.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55480





More information about the llvm-commits mailing list