[PATCH] D26224: NewGVN

[Davide Italiano via llvm-commits]

davide added a comment.

In https://reviews.llvm.org/D26224#600689, @jwhowarth wrote:

> Can you also post on Phabricator a mockup of a patch to enable the newgvn pass by default in the trunk clang compiler in order to simplify end-user testing? I noticed that the current clang compiler doesn't seem to enable the current gvn pass by default.


Sure, I'll make one. I'm not sure what you mean when you say that the current clang compiler doesn't call the current GVN. In `EmitAssemblyHelper::CreatePasses`, we call inside LLVM, `PMBuilder.populateModulePassManager()` which creates a pass pipeline containing NewGVN (see `Transforms/IPO/PassManagerBuilder.cpp`).


https://reviews.llvm.org/D26224