Re: [PATCH] you can use this patch to profile cygwin

[jojelino <jojelino at gmail dot com>]

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

Attachment: ChangeLog
Description: Text document

Attachment: winsup-makefile.diff
Description: Text document

Attachment: winsup-profile-support.diff
Description: Text document