[PATCH] D26224: NewGVN

[Daniel Berlin via llvm-commits]

Davide, i fixed this on my branch :)


On Mon, Nov 21, 2016 at 4:52 AM, James Molloy <james.molloy at arm.com> wrote:

> jmolloy added a comment.
>
> Hi Davide,
>
> A high level comment for an issue I've just run into when rebasing GVNSink
> on top of NewGVN - you've defined LoadExpression::equals and
> StoreExpression::equals in the header, which gives multiple definition
> errors (for ::equals and the vtables) if any other file tries to #include
> <GVNExpression.h>.
>
> Cheers,
>
> James
>
>
> https://reviews.llvm.org/D26224
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161121/725368c1/attachment.html>