Re: No need for object file removal before compilation

2008-04-18 Thread Ralf Wildenhues
* Bob Friesenhahn wrote on Sat, Apr 12, 2008 at 05:51:28PM CEST: The patch looks ok to me. We shall see about the actual performance improvement. Thanks, applied. Cheers, Ralf 2008-04-12 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/config/ltmain.m4sh (func_mode_compile): Avoid

Re: No need for object file removal before compilation

2008-04-12 Thread Ralf Wildenhues
OK, this issue is much more intricate than it looked like at first. First, the existing code had some bugs: - it removed files too often. After the first compile, we *know* that there are no files to stumble over; if the compiler doesn't know -c -o, then we have moved away $output_obj

Re: No need for object file removal before compilation

2008-04-12 Thread Ralf Wildenhues
* Ralf Wildenhues wrote on Sat, Apr 12, 2008 at 08:56:24AM CEST: OK for the updated one? It makes for 5% faster libtool --mode=compile gcc -c a.c I should be more precise: the above holds for an empty file a.c, measured using a script that calls the above 50 times, pre and post patch. I

Re: No need for object file removal before compilation

2008-04-12 Thread Bob Friesenhahn
On Sat, 12 Apr 2008, Ralf Wildenhues wrote: Then, while I have some doubts about the practical applicability of Bob's scenario, I left the code so that it would be supported: all Bob's scenarios are rarely practical. I would have been inclined to ok this one. OK for the updated one? It

Re: No need for object file removal before compilation

2008-04-10 Thread Peter O'Gorman
Bob Friesenhahn wrote: On Thu, 10 Apr 2008, Ralf Wildenhues wrote: OK to apply? I fail to see why it's necessary to remove these files. If they existed from a previous compile, they are outdated, so what. But I may be overlooking something here. Removing the object file in advance

No need for object file removal before compilation

2008-04-09 Thread Ralf Wildenhues
OK to apply? I fail to see why it's necessary to remove these files. If they existed from a previous compile, they are outdated, so what. But I may be overlooking something here. Thanks, Ralf 2008-04-10 Ralf Wildenhues [EMAIL PROTECTED] * libltdl/config/ltmain.m4sh