[PATCH] D33584: Remove a quadratic behavior in assert-enabled builds

[Adam Nemet via Phabricator via llvm-commits]

anemet added a comment.

In https://reviews.llvm.org/D33584#766183, @sanjoy wrote:

> In https://reviews.llvm.org/D33584#766182, @anemet wrote:
>
> > I certainly agree that if we're not returning false for the non-unique edge case then that will cause bugs later on.
> >
> > Can I add unit-tests for edge-domination somehow?  I am trying to test this with allowing non-unique edges in GVN but that won't fly as regression test.
>
>
> I'd just go with a regular C++ test case in unittests/


OK, will do.  Thanks, guys!


https://reviews.llvm.org/D33584