[PATCH] D147056: [SLP][AArch64] Add test to check for the vectorization of fshl

[Florian Hahn via Phabricator via llvm-commits]

fhahn accepted this revision.
fhahn added a comment.
This revision is now accepted and ready to land.

LGTM, thanks! Maybe drop the `-vectorize` part from the name :)



================
Comment at: llvm/test/Transforms/SLPVectorizer/AArch64/fshl-vectorize.ll:3
+; RUN: opt -mtriple=arm64-apple-ios -S -passes=slp-vectorizer < %s | FileCheck %s
+; fshl instruction cost model is an overestimate causing this test to vectorize when it is not beneficial to do so.
+
----------------
Can you add a newline before and remove the newline before the function definition?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147056