[PATCH] D124309: [SLP] Steer for the best chance in tryToVectorize() when rooting with binary ops.

Fri Apr 22 16:19:09 PDT 2022

ABataev added a comment.

Hi Valery, thanks for the patch. This looks good! Could you split it into 2 patches: one that renames lookahead heuristics (NFC) and the second (functional) one?

================
Comment at: llvm/test/Transforms/SLPVectorizer/AArch64/invalid_type.ll:7
 ; This test check that slp vectorizer is not trying to vectorize instructions already vectorized.
-; CHECK: remark: <unknown>:0:0: Cannot SLP vectorize list: type <16 x i8> is unsupported by vectorizer

----------------
Do you need to update this test?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124309