[PATCH] you can use this patch to profile cygwin

[jojelino]

On 2011-08-11 PM 12:49, jojelino wrote:
 > On 2011-08-01 AM 2:30, Christopher Faylor wrote:
 >> And, as usual, the formatting is completely off so it couldn't be
 >> incorporated into the source code as is regardless.
 >>
 > But i don't know where the formatting guideline is. may i ask you giving
 > me the link?
 >
The formatting is revised to adopt GNU code style (using Eclipse CDT).
and if you want make new-cygwin1.dll to profiling itself, please command

cd i686-pc-cygwin/winsup/cygwin;rm *.o;make profile=1

it would work.
and there was some issue to winsup/Makefile.common, simply expanding 
ALL_CFLAGS resulted in CFLAGS are not expanded to ALL_CFLAGS. in cases 
of gmon.c,

gcc -L/tmp/winsup/i686-pc-cygwin/winsup 
-L/tmp/winsup/i686-pc-cygwin/winsup/cygwin 
-L/tmp/winsup/i686-pc-cygwin/winsup/w32api/lib -isystem 
/tmp/winsup/winsup/include -isystem /tmp/winsup/winsup/cygwin/include 
-isystem /tmp/winsup/winsup/w32api/include 
-B/tmp/winsup/i686-pc-cygwin/newlib/ -isystem 
/tmp/winsup/i686-pc-cygwin/newlib/targ-include -isystem 
/tmp/winsup/newlib/libc/include    -c     -Wall -Wstrict-aliasing 
-Wwrite-strings -fno-common -pipe -fbuiltin -fmessage-length=0  -I. 
-I/tmp/winsup/winsup/cygwin  -I/tmp/winsup/winsup/w32api/include 
-I../../.././winsup/cygwin/config/i386 
-I/usr/lib/gcc/i686-pc-cygwin/4.7.0/include -o ./gmon.o 
/tmp/winsup/winsup/cygwin/gmon.c

after replacing simple expansion to recursively expansion, CFLAGS are 
expanded properly. is it intended one?

gcc -L/tmp/winsup/i686-pc-cygwin/winsup 
-L/tmp/winsup/i686-pc-cygwin/winsup/cygwin 
-L/tmp/winsup/i686-pc-cygwin/winsup/w32api/lib -isystem 
/tmp/winsup/winsup/include -isystem /tmp/winsup/winsup/cygwin/include 
-isystem /tmp/winsup/winsup/w32api/include 
-B/tmp/winsup/i686-pc-cygwin/newlib/ -isystem 
/tmp/winsup/i686-pc-cygwin/newlib/targ-include -isystem 
/tmp/winsup/newlib/libc/include    -c  -DHAVE_CONFIG_H  -O4 
-fomit-frame-pointer -mstackrealign -march=core2 -g -mfpmath=sse -MMD 
-fmerge-constants -ftracer -mno-use-libstdc-wrappers  -Wall 
-Wstrict-aliasing -Wwrite-strings -fno-common -pipe -fbuiltin 
-fmessage-length=0  -I.  -I/tmp/winsup/winsup/cygwin 
-I/tmp/winsup/winsup/w32api/include 
-I../../.././winsup/cygwin/config/i386 
-I/usr/lib/gcc/i686-pc-cygwin/4.7.0/include -o ./gmon.o 
/tmp/winsup/winsup/cygwin/gmon.c



-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: ChangeLog
URL: <http://cygwin.com/pipermail/cygwin-developers/attachments/20110819/c5a73636/attachment.ksh>
-------------- next part --------------
Index: winsup/Makefile.common
===================================================================
RCS file: /cvs/src/src/winsup/Makefile.common,v
retrieving revision 1.56
diff -u -r1.56 Makefile.common
--- winsup/Makefile.common      7 Jul 2009 20:12:43 -0000       1.56
+++ winsup/Makefile.common      19 Aug 2011 20:22:40 -0000
@@ -113,7 +113,7 @@
 LIBM:=$(newlib_build)/libm/libm.a
 CRT0:=$(cygwin_build)/crt0.o

-ALL_CFLAGS:=$(DEFS) $(MALLOC_DEBUG) $(CFLAGS) $(GCC_DEFAULT_OPTIONS)
+ALL_CFLAGS=$(DEFS) $(MALLOC_DEBUG) $(CFLAGS) $(GCC_DEFAULT_OPTIONS)
 ALL_CXXFLAGS=$(DEFS) $(MALLOC_DEBUG) $(CXXFLAGS) $(GCC_DEFAULT_OPTIONS)

 ifndef PREPROCESS
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: winsup-profile-support.diff
URL: <http://cygwin.com/pipermail/cygwin-developers/attachments/20110819/c5a73636/attachment-0001.ksh>