* [PATCH][LTO] Fix PR41858
@ 2009-10-29 14:31 Toon Moene
2009-10-29 14:44 ` Richard Guenther
0 siblings, 1 reply; 4+ messages in thread
From: Toon Moene @ 2009-10-29 14:31 UTC (permalink / raw)
To: rguenther; +Cc: gcc mailing list
You wrote:
> I refrained from adding a 4000 file testcase ;)
Never mind - I have one. I didn't understand why lto1 said this:
/usr/snp/lib/gcc/x86_64-unknown-linux-gnu/4.5.0/../../../../x86_64-unknown-linux
-gnu/bin/ld: fatal error: could not open/create temporary file
while dealing with the largest executable of our weather forecasting system.
[ BTW, you could just say ulimit -n 10 before running the test :-) ]
--
Toon Moene, KNMI (Weer/Onderzoek), The Netherlands
Phone: +31 30 2206443; e-mail: moene@knmi.nl
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH][LTO] Fix PR41858
2009-10-29 14:31 [PATCH][LTO] Fix PR41858 Toon Moene
@ 2009-10-29 14:44 ` Richard Guenther
2009-10-29 14:45 ` Toon Moene
0 siblings, 1 reply; 4+ messages in thread
From: Richard Guenther @ 2009-10-29 14:44 UTC (permalink / raw)
To: Toon Moene; +Cc: gcc mailing list
On Thu, 29 Oct 2009, Toon Moene wrote:
> You wrote:
>
> > I refrained from adding a 4000 file testcase ;)
>
> Never mind - I have one. I didn't understand why lto1 said this:
>
> /usr/snp/lib/gcc/x86_64-unknown-linux-gnu/4.5.0/../../../../x86_64-unknown-linux
> -gnu/bin/ld: fatal error: could not open/create temporary file
>
> while dealing with the largest executable of our weather forecasting system.
>
> [ BTW, you could just say ulimit -n 10 before running the test :-) ]
Heh, indeed. Btw, I didn't yet hit the above error. Are you using
the linker plugin?
Richard.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH][LTO] Fix PR41858
2009-10-29 14:44 ` Richard Guenther
@ 2009-10-29 14:45 ` Toon Moene
2009-10-29 15:09 ` Richard Guenther
0 siblings, 1 reply; 4+ messages in thread
From: Toon Moene @ 2009-10-29 14:45 UTC (permalink / raw)
To: Richard Guenther; +Cc: gcc mailing list
Richard Guenther wrote:
> On Thu, 29 Oct 2009, Toon Moene wrote:
>
>> You wrote:
>>
>>> I refrained from adding a 4000 file testcase ;)
>> Never mind - I have one. I didn't understand why lto1 said this:
>>
>> /usr/snp/lib/gcc/x86_64-unknown-linux-gnu/4.5.0/../../../../x86_64-unknown-linux
>> -gnu/bin/ld: fatal error: could not open/create temporary file
>>
>> while dealing with the largest executable of our weather forecasting system.
>>
>> [ BTW, you could just say ulimit -n 10 before running the test :-) ]
>
> Heh, indeed. Btw, I didn't yet hit the above error. Are you using
> the linker plugin?
Indeed. Your fix *does* work for that case, no ?
--
Toon Moene, KNMI (Weer/Onderzoek), The Netherlands
Phone: +31 30 2206443; e-mail: moene@knmi.nl
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH][LTO] Fix PR41858
2009-10-29 14:45 ` Toon Moene
@ 2009-10-29 15:09 ` Richard Guenther
0 siblings, 0 replies; 4+ messages in thread
From: Richard Guenther @ 2009-10-29 15:09 UTC (permalink / raw)
To: Toon Moene; +Cc: gcc mailing list
On Thu, 29 Oct 2009, Toon Moene wrote:
> Richard Guenther wrote:
>
> > On Thu, 29 Oct 2009, Toon Moene wrote:
> >
> > > You wrote:
> > >
> > > > I refrained from adding a 4000 file testcase ;)
> > > Never mind - I have one. I didn't understand why lto1 said this:
> > >
> > > /usr/snp/lib/gcc/x86_64-unknown-linux-gnu/4.5.0/../../../../x86_64-unknown-linux
> > > -gnu/bin/ld: fatal error: could not open/create temporary file
> > >
> > > while dealing with the largest executable of our weather forecasting
> > > system.
> > >
> > > [ BTW, you could just say ulimit -n 10 before running the test :-) ]
> >
> > Heh, indeed. Btw, I didn't yet hit the above error. Are you using
> > the linker plugin?
>
> Indeed. Your fix *does* work for that case, no ?
Well, it fixes a leak in lto1 which affected both using the linker
plugin and not. The above error seems to be from ld instead or the
linker plugin (or maybe collect2).
Richard.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2009-10-29 14:44 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-10-29 14:31 [PATCH][LTO] Fix PR41858 Toon Moene
2009-10-29 14:44 ` Richard Guenther
2009-10-29 14:45 ` Toon Moene
2009-10-29 15:09 ` Richard Guenther
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).