Basant Kukreja wrote:
> 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.
>
>
okay then ..
> 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
>>>
>>>
