On Tue, 15 Jan 2019 at 14:33, Richard Biener <rguent...@suse.de> wrote: > > On Tue, 15 Jan 2019, Jakub Jelinek wrote: > > > On Tue, Jan 15, 2019 at 12:29:21PM +0530, Prathamesh Kulkarni wrote: > > > 2019-01-15 Richard Biener <rguent...@suse.de> > > > Prathamesh Kulkarni <prathamesh.kulka...@linaro.org> > > > > > > PR ipa/88378 > > > > Wrong PR number. > > > > > * ipa-pure-const.c (malloc_candidate_p_1): Add parameter visited and > > > return true if SSA_NAME is already marked in visited bitmap. > > > * (malloc_candidate_p): Pass visited to malloc_candidate_p_1. > > > > > > testsuite/ > > > * g++.dg/ipa/pr88788.C: New test. > > > > The testcase looks way too large to me, can't we reduce it? > > > > delta/creduce with a hanging compiler isn't fast (one would need to use > > timeout), but e.g. in #c1 you have a patch to get an ICE instead of hang if > > it happens. > > Or just omit the testcase - even with the bitmap we can end up recursing > very many times thus the ICE can happen spuriously. I committed patch in r267933 omitting the test-case. Thanks Richard and Jakub for help with the bug!
Thanks, Prathamesh > > Richard.