Re: [PHP-DEV] --enable-debug and GCC 4.8 / -Og

2013-07-08 Thread Rainer Jung
On 07.07.2013 16:31, Sebastian Bergmann wrote:
>  From the release notes of GCC 4.8:
> 
>"A new general optimization level, -Og, has been introduced. It
>addresses the need for fast compilation and a superior debugging
>experience while providing a reasonable level of runtime performance.
>Overall experience for development should be better than the default
>optimization level -O0."
> 
>  Currently "--enable-debug" sets "-O0". I think it would make sense to
>  use "-Og" instead when GCC >= 4.8.0 is used.
> 
>  What do you think?

There's a recent short discussion about -Og at

http://gcc.gnu.org/ml/gcc/2013-07/msg1.html

Regards,

Rainer


-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-DEV] --enable-debug and GCC 4.8 / -Og

2013-07-08 Thread Sebastian Bergmann
On 07/08/2013 06:54 PM, Christopher Jones wrote:
> Your test logic needs fixing.  It should be like: "if ((major > 4) ||
> (major == 4 && minor >= 8))"

 Thank you, Chris, for pointing this out. I have updated the gist.

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-DEV] --enable-debug and GCC 4.8 / -Og

2013-07-08 Thread Christopher Jones



On 07/07/2013 07:55 AM, Sebastian Bergmann wrote:

On 07/07/2013 04:31 PM, Sebastian Bergmann wrote:

Currently "--enable-debug" sets "-O0". I think it would make sense to
use "-Og" instead when GCC >= 4.8.0 is used.


  https://gist.github.com/sebastianbergmann/5943708 is an attempt to
  implement this. Only tested this on Fedora 19.



Seems reasonable.

Your test logic needs fixing.  It should be like: "if ((major > 4) || (major == 4 && 
minor >= 8))"

Chris

--
christopher.jo...@oracle.com  http://twitter.com/ghrd
Free PHP & Oracle book:
http://www.oracle.com/technetwork/topics/php/underground-php-oracle-manual-098250.html

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php