Earlier I had posted about testing O+ with Apache using Symfony's PhpUnit tests... I had an issue where it seems that test methods (which PhpUnit accesses using ReflectionMethod) were getting optimized out and replaced with an empty method (so all PhpUnit tests passed). I have now written a PHPT to try to reproduce this issue (attached in case it helps), which passes/doesn’t have this issue.
I don’t think this issue is full resolved/closed, but it’s a low-priority edge case that can be fixed later. Symfony runs with O+(see O+ README for proper config)…. This issue doesn’t affect a normal Symfony app running on Apache with O+... users aren't likely to have this problem. If users do have this problem, I have found that turning off all optimization passes except pass 3 and pass 10 (using the optimization_level directive) seems to fix this problem. The reason I found this problem is that I run multiple PhpUnit tests on each Apache instance, one test is run at a time, but multiple tests are run against each instance/process during its lifetime. This is a little more accurate (apache doesn’t reboot between each HTTP request) and makes testing much faster, otherwise I have to start and stop apache between each test, which is really slow. I have confirmed this by restarting apache between each PhpUnit test, and I get about the same errors and failures as I do without O+. Additionally, I get a bunch of crashes, which I will be investigating further. Regardless of the crashes, this issue is not critical/low priority now since all the PhpUnit tests actually now run (reflectionmethods aren’t getting optimized out). Regards -Matt > Date: Sat, 2 Mar 2013 09:43:34 +0100 > From: tyr...@gmail.com > To: z...@zend.com > CC: internals@lists.php.net; dmi...@zend.com > Subject: Re: [PHP-DEV] Current Status of O+ on Windows > > On Sat, Mar 2, 2013 at 9:39 AM, Zeev Suraski <z...@zend.com> wrote: > > > > -----Original Message----- > > > From: Ferenc Kovacs [mailto:tyr...@gmail.com] > > > Sent: Saturday, March 02, 2013 10:15 AM > > > To: Pierre Joye; Dmitry Stogov > > > Cc: Christopher Jones; Matt Ficken; internals@lists.php.net > > > Subject: Re: [PHP-DEV] Current Status of O+ on Windows > > > > > > > > Did you experiment with any options? > > > > > Putting the date of your O+ snapshot would also be handy. > > > > > > > > Latest from here are used: > > > > http://windows.php.net/downloads/pecl/snaps/Optimizer/7.0.0-dev/ > > > > > > > > Dates are included. It would be nice to have it in the report as well, > > > > but we always use latest revision. It would however be much easier if > > > > there were PECL releases. > > > > > > > > > > > +1 > > > Dmitry, is there any objection against creating a pecl release? > > > Could you tag the first version? > > > > The current vote that's going on right now deals with putting the extension > > into PHP itself. If that happens (which seems awfully likely at this > > point), why do we need it in PECL? We'd very much rather invest our very > > limited cycles into the code itself. We're probably roughly a week away > > from having builds as a part of the PHP 5.5 snaps. > > > > Zeev > > > > I see. > so no O+ for <5.5? > having a pecl release would be a small amount of work, which I would glad > to help with. > > -- > Ferenc Kovács > @Tyr43l - http://tyrael.hu
-- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
