This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin 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: [PATCH] enable -finline-functions optimization


On Wed, Apr 09, 2003 at 11:03:42AM +0200, Thomas Pfaff wrote:
>
>This patch enables inline optimization for the c++ source files
>in winsup/cygwin.
>
>I tried several attributes for std_dll_init, wsock_init and
>unused_sig_wrapper without success, the only working solution was to
>change the functions from static to global to avoid its removal.
>
>And the new_muto in the pwdgrp constructors can not be inlined for more
>than one instance.
>
>To enable -finline-functions optimization run "make CYGINLINE=1".
>
>The performance win may vary, at least the rwlock7 test runs about 15%
>faster.
>
>I am running cygwin with this patch about one week now, seems to be
>stable.
>
>2003-04-09  Thomas Pfaff  <tpfaff at gmx dot net>
>
>	* Makefile.in: Add finline-functions optimization to CXXFLAGS.
>	* autoload.cc (std_dll_init): Change from static to global.

Sorry but I'm not going to change something that should be a valid static into
a global just to accommodate the compiler.  There should be compiler attributes
which allow this to behave normally.

Also, minor point of order:  This a [RFA], not a [PATCH].

cgf


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