[llvm] 8e67982 - [NewGVN] Add test cases for remaining known issues.

[Florian Hahn via llvm-commits]

On Wed, Jul 29, 2020 at 11:39 AM David Zarzycki <dave at znu.io> wrote:

> Hi Florian,
>
> This seems to have exposed a latent bug. In particular,
> llvm/test/Transforms/NewGVN/todo-pr37121-seens-this-value-a-lot.ll spins
> out of control until the test times out as `opt` consumes hundreds of
> gigabytes of RAM.
>
> My setup: `opt` built release+no-assert via Fedora's build of clang-10 on
> Fedora 32 *(x86-64).
>

Thanks for letting me know. The test is marked as XFAIL, but it requires
asserts to fail.  The tests really should only run with assertions enabled.
They should be guarded by `REQUIRES: asserts`

I can add that.

Cheers,
Florian
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200729/b68c07b2/attachment.html>