This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [gfortran,patch] PR fortran/28163: Don't call library function for string copying


On Sun, Jun 25, 2006 at 05:46:11PM +0200, FX Coudert wrote:
> >I'm curious if this should be selectable via something like
> >-minline-all-stringops resp. -f{no-}builtin-whatever?
> >
> >While it may give a nice performance boost, it most likely is something
> >e.g. -Os doesn't like, fwiw.
> 
> You're right about -Os, though I don't think it needs an option all for 
> itself. Also, we probably won't be inlining all string operations, 
> because some would be a real pain to maintain if they were inside the 
> front-end.
> 
> What about simply call the library version if -Os is specified, i.e. 
> make the new code conditional on optimize_size? Would that seem acceptable?
> 

What are the sizes of aermod with and without your patch?
Checking Polyhedron's posted benchmark results suggest that
a 27% speed up is an important optizimation regardless of
the use of -Os.
 
-- 
Steve


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]