[PATCH] D28907: [SLP] Fix for PR30787: Failure to beneficially vectorize 'copyable' elements in integer binary ops.

Fri Dec 7 07:44:13 PST 2018

dtemirbulatov marked an inline comment as done.
dtemirbulatov added inline comments.

================
Comment at: test/Transforms/SLPVectorizer/X86/invoke.ll:3
+; RUN: opt -S -slp-vectorizer -slp-vectorizer -mcpu=bdver1 < %s | FileCheck %s
+
+target datalayout = "e-m:e-i64:64-f80:128-n8:16:32:64-S128"
----------------
lebedev.ri wrote:
> dtemirbulatov wrote:
> > dtemirbulatov wrote:
> > > lebedev.ri wrote:
> > > > Unless this currently crashes, precommit?
> > > yes, thanks, I will remove this line.
> > Oh, No crash currently on pre-commit, I will remove the whole test.
> What i have meant to say is, unless this test file currently causes a crash, it would be best to commit it now, thus the diff will show the diff, and not a whole new test file.
Ok

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

https://reviews.llvm.org/D28907