[PATCH] D146958: [SCEV] Do not plant SCEV checks unnecessarily

[Paul Osmialowski via Phabricator via llvm-commits]

pawosm01 marked 2 inline comments as done.
pawosm01 added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/vector-no-scevcheck.ll:1
+; REQUIRES: aarch64-registered-target
+; RUN: opt -passes=loop-vectorize -force-vector-interleave=1 -S < %s | FileCheck %s
----------------
ABataev wrote:
> Could you precommit the test?
will do soon(ish)



================
Comment at: llvm/test/Transforms/LoopVectorize/vector-no-scevcheck.ll:4-5
+
+target datalayout = "e-m:e-i8:8:32-i16:16:32-i64:64-i128:128-n32:64-S128"
+target triple = "aarch64-unknown-linux-gnu"
+
----------------
ABataev wrote:
> Pass these as arguments of opt call.
Actually, I see no good reason for having them. No need for limiting it to AArch64.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146958