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

[Florian Hahn via llvm-commits]

Should be fixed by 2aa2c40d23740331e01c4eeec849775ce9c33f95.

Please let me know if you still see the issue.

On Wed, Jul 29, 2020 at 12:27 PM Florian Hahn <flo at fhahn.com> wrote:

>
>
> 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
>
>

-- 

http://www.fhahn.com/
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200729/e028e646/attachment.html>