Edit report at https://bugs.php.net/bug.php?id=70173&edit=1
ID: 70173
Updated by: laruence@php.net
Reported by: rainer dot jung at kippdata dot de
Summary: ZVAL_COPY_VALUE_EX broken for 32bit Solaris Sparc
-Status: Analyzed
+Status: Closed
Type: Bug
Package: Scripting Engine problem
Operating System: Solaris 10 Sparc
PHP Version: 7.0.0beta3
Block user comment: N
Private report: N
New Comment:
Automatic comment on behalf of laruence
Revision: http://git.php.net/?p=php-src.git;a=commit;h=89940993f0f60415e539b05dbec2c24fa79bca81
Log: Fixed Bug #70173 (ZVAL_COPY_VALUE_EX broken for 32bit Solaris Sparc)
Previous Comments:
------------------------------------------------------------------------
[2015-08-09 18:10:27] rainer dot jung at kippdata dot de
PR tested. Build succeeds, running test suite shows good results including the new test. Looks good
to me for merging. Thanks a bunch.
------------------------------------------------------------------------
[2015-08-09 14:28:05] cmb@php.net
You can find it on <https://github.com/php/php-src/pull/1464>.
(It's also linked further above in this ticket in the "Pull
Requests" section.)
------------------------------------------------------------------------
[2015-08-09 14:26:23] rainer dot jung at kippdata dot de
Ah, OK, found it at Github. I will add the change to my build and restest. Building and testing will
take a few hours. Thanks for the fast reaction!
------------------------------------------------------------------------
[2015-08-09 14:23:03] rainer dot jung at kippdata dot de
Where do I find "PR #1464"? Is there a typo (the number is very small).
I can test any suggested change.
------------------------------------------------------------------------
[2015-08-09 13:15:57] cmb@php.net
Thanks, Rainer, for the thorough analysis. AIUI the portable data
layout (ZEND_ENDIAN_LOHI) allows for more efficient operations[1].
PR #1464 is supposed to solve the issue. I do not have a
big-endian machine at hand, so I can't test it, though.
[1] <https://github.com/php/php-src/commit/d8099d0468426dbee59f540048376653535270ce>
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=70173
--
Edit this bug report at https://bugs.php.net/bug.php?id=70173&edit=1