[PATCH] D105587: [Instcombine]Transform reduction+(sext/zext(<n x i1>) to <n x im>) to [-]zext/trunc(ctpop(bitcast <n x i1> to in)) to im.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 8 07:25:36 PDT 2021


ABataev added inline comments.


================
Comment at: llvm/test/Transforms/InstCombine/reduction-add-sext-zext-i1.ll:1
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
+; RUN: opt < %s -instcombine -S | FileCheck %s
----------------
lebedev.ri wrote:
> Please precommit
Will do.


================
Comment at: llvm/test/Transforms/InstCombine/reduction-add-sext-zext-i1.ll:65
+}
+
+declare i1 @llvm.vector.reduce.add.v8i32(<8 x i1> %a)
----------------
lebedev.ri wrote:
> Could use a test where the `zext`/`sext` has extra use and doesn't go away.
Will add


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105587



More information about the llvm-commits mailing list