If we were to contribute this patch to community then yes, patch should check if the compiler is studio compiler or not. But the patch is Solaris specific patch so I didn't want to add complexity into the patch.
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 >>
