[PATCH] D62308: [AArch64] support neon_sshl and neon_ushl in performIntrinsicCombine.

Mon Sep 23 02:38:47 PDT 2019

fhahn added a comment.

In D62308#1676800 <https://reviews.llvm.org/D62308#1676800>, @anemet wrote:

> In D62308#1676691 <https://reviews.llvm.org/D62308#1676691>, @fhahn wrote:
>
> > I'll submit a separate patch for turning ushl -> shl, if the shift is all constant.
>
>
> ... and sshl -> shl?

Yep, let's look at that separately.

================
Comment at: llvm/test/CodeGen/AArch64/arm64-vshift.ll:1318
+
+; FIXME: unnecessary sshll.4s v0, v0, #0?
+define <4 x i32> @neon.sshll4s_neg_constant_shift(<4 x i16>* %A) nounwind {
----------------
anemet wrote:
> Isn't it used for the extensions?
Yeah, I initially thought there might be a long version of sshr, but looks like there is not

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62308