[PATCH] D30478: Move memory coercion functions from GVN.cpp to VNCoercion.cpp so they can be shared between GVN and NewGVN.

[Daniel Berlin via llvm-commits]

Whoops, thought i had one :)
I'll fix.


On Sun, Mar 5, 2017 at 3:35 PM, Davide Italiano via Phabricator <
reviews at reviews.llvm.org> wrote:

> davide accepted this revision.
> davide added a comment.
> This revision is now accepted and ready to land.
>
> LGTM, with the inline comment addressed. Thanks!
>
>
>
> ================
> Comment at: lib/Transforms/Utils/VNCoercion.cpp:1-6
> +#include "llvm/Transforms/Utils/VNCoercion.h"
> +#include "llvm/Analysis/AliasAnalysis.h"
> +#include "llvm/Analysis/ConstantFolding.h"
> +#include "llvm/Analysis/MemoryDependenceAnalysis.h"
> +#include "llvm/Analysis/ValueTracking.h"
> +#include "llvm/IR/IRBuilder.h"
> ----------------
> This probably needs an header comment with the license etc..
>
>
> https://reviews.llvm.org/D30478
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170305/35fca468/attachment.html>