[PATCH] D22639: Add flag to PassManagerBuilder to disable GVN Hoist Pass.

[Alina Sbirlea via llvm-commits]

I would guess that yes? just my optinion.
GVN is disabled by default, hence the need to add `-enable-gvn-hoist` for
any opt level >=O1 (vs DCE which is enabled by default). On the other hand
explicitly adding `-gvn-hoist` should also work, just like adding `-dce`.

On Fri, Jul 29, 2016 at 8:34 AM, Paul Robinson <paul.robinson at sony.com>
wrote:

> probinson added a subscriber: probinson.
> probinson added a comment.
>
> This means to enable the pass in opt I need to do both `-enable-gvn-hoist`
> and `-O1` which is not usual for opt.  (For example I need just `-dce` to
> get DCE.)  Is that the intent?
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D22639
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160729/6fa4a51b/attachment.html>