[PATCH] D100485: [AArch64][SVE] Fix crash with icmp+select

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 14 13:08:07 PDT 2021


sdesmalen added a comment.

Can you also add support for predicates?

This currently also fails:

  define <vscale x 16 x i1> @icmp_select_nxv16i1(<vscale x 16 x i1> %a, <vscale x 16 x i1> %b, i64 %x0) {
    %mask = icmp eq i64 %x0, 0
    %sel = select i1 %mask, <vscale x 16 x i1> %a, <vscale x 16 x i1> %b
    ret <vscale x 16 x i1> %sel
  }



================
Comment at: llvm/test/CodeGen/AArch64/select-sve.ll:550
+
+define <vscale x 16 x i64> @icmp_select_nxv16i64(<vscale x 16 x i64> %a, <vscale x 16 x i64> %b, i64 %x0) {
+; CHECK-LABEL: icmp_select_nxv16i64
----------------
I don't think testing these wider types is necessary, as they will just rely on splitting the `select`, for which we already have tests.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100485



More information about the llvm-commits mailing list