[PATCH] D156532: [Pipelines] Perform hoisting prior to GVN

[Nikita Popov via Phabricator via llvm-commits]

nikic added inline comments.


================
Comment at: llvm/test/Transforms/PhaseOrdering/gvn-replacement-vs-hoist.ll:2
 ; NOTE: Assertions have been autogenerated by utils/update_test_checks.py UTC_ARGS: --version 2
 ; RUN: opt -S -O3 < %s | FileCheck %s
 
----------------
aeubanks wrote:
> artagnon wrote:
> > I just noticed that you're using the old PM. If you change this to `opt -S -passes='default<O3>'`, this example will be vectorized as well.
> they should be equivalent now
This test doesn't use a target triple to keep it target-independent. If you specify one, then yes, it does get vectorized (well, depending on target of course).


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

https://reviews.llvm.org/D156532