Thanks Sriram for your comments. I have revised the patch. It now uses Studio flags when SUNCC is true.
Regards, Basant. On Mon, Nov 09, 2009 at 10:06:03AM -0800, Sriram Natarajan wrote: > for the configure.in patch to be generic, i think it would be better if > you check the CC to be Sun Studio's CC before you add those flags.. > > Basant Kukreja wrote: >> Hi, >> Kindly review the webrev for the bug >> 6899209 : Compile zend_execute.c with -W2,-Ainline:inc=100 for performance >> gains >> >> http://cr.opensolaris.org/~basantk/6899209/webrev/ >> >> Thanks, >> Basant. >> >> _______________________________________________ >> >> >> webstack-discuss mailing list >> webstack-discuss at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/webstack-discuss >>