On 10/4/19 1:43 PM, co...@sdf.org wrote: > On Tue, Oct 01, 2019 at 01:26:16PM -0600, Jeff Law wrote: >> On 9/30/19 2:45 PM, co...@sdf.org wrote: >>> On Mon, Sep 30, 2019 at 11:46:24AM -0400, Vladimir Makarov wrote: >>>> Yes, the patch is mostly ok. You can commit it into the trunk after >>>> applying changes mentioned below. If you do not have a write access, let me >>>> know I'll commit the patch by myself. >>> >>> I don't have commit access. It would be nice if you committed it :) >> I took care of the nits and committed the patch. >> >> >>> >>>> It would be nice to add a small test too. But it is not obligatory for >>>> this >>>> case as the patch is obvious and it might be hard to create a small test to >>>> reproduce the bug. >>> >>> I have the C code that produces this failure. I can creduce it, but I'm >>> not sure there's a relationship between it and the bug. >>> Doing unrelated changes (adding instruction scheduling) to vax also hid it. >>> >>> Is this kind of test still valuable? >> Often they are. >> >> jeff > > Here's the two tests I used. It might be too machine-made. > One is in the vax specific directory since it needed -fno-pic. > > > 2019-10-04 Maya Rashish <co...@sdf.org> > * gcc.c-torture/compile/pr85401-2.c: New test. > * gcc.target/vax/pr85401-1.c: New test. ISTM that both should be in c-torture. We can use dg-* things in there too.
jeff