[PATCH] D67359: [IVUser] Limit the iterations to check whether a loop has dedicated exits for extreme large case

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 23 17:03:04 PDT 2019


xbolva00 added a comment.

In D67359#1679990 <https://reviews.llvm.org/D67359#1679990>, @wmi wrote:

> In D67359#1679979 <https://reviews.llvm.org/D67359#1679979>, @xbolva00 wrote:
>
> > Probably just put it here:
> >
> > https://github.com/llvm/llvm-test-suite/tree/master/SingleSource/Regression/C
>
>
> Thanks, I can put the test there, but I am not sure how large should I increase the size of the test to so it will definitely timeout without the patch, and do I need to consider the test takes different time to compile on different machines? I remember there is a fixed compile time limit to build a file in testsuite but I don't remember how long it is. Do you happen to know that?


I dont know :( but 2s is probably okay. Maybe @fhahn can you help here..


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67359/new/

https://reviews.llvm.org/D67359





More information about the llvm-commits mailing list