Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:67408
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.219.51 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <CAESVnVov7hyt=AW+XwB2z6Jf3870bJO5DcnFTb7o66FN3Ds1TA@mail.gmail.com>
References: <CAF+90c_PPTajBqs636f373DKF=hotrK1ObTojddxx=bxC3g16w@mail.gmail.com>
	<518FD653.6070705@daylessday.org>
	<CAF+90c_qj9_GcqhYSBRnbYLBJ-31iBSUTdnVn8owWJxpzqRedQ@mail.gmail.com>
	<CAESVnVov7hyt=AW+XwB2z6Jf3870bJO5DcnFTb7o66FN3Ds1TA@mail.gmail.com>
Date: Sun, 12 May 2013 22:07:42 +0200
Message-ID: <CAF+90c9d-R9PqPAKuFLpc3yHE5sVMbTYScw1UH0jVrH5zhBaoA@mail.gmail.com>
To: Sara Golemon <pollita@php.net>
Cc: Antony Dovgal <tony@daylessday.org>, PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=047d7b41408ee4d49f04dc8af3da
Subject: Re: [PHP-DEV] [RFC] Internal operator overloading and GMP improvements
From: nikita.ppv@gmail.com (Nikita Popov)

--047d7b41408ee4d49f04dc8af3da
Content-Type: text/plain; charset=ISO-8859-1

On Sun, May 12, 2013 at 9:57 PM, Sara Golemon <pollita@php.net> wrote:

> Are we ignoring the ZEND_IS_SMALLER issue?
>
Not ignoring it :) It's mentioned in one sentence: "The operators >, >=, [...]
are indirectly supported by the following compiler transformations: [...]"

if ($gmp > 123) { ... }
>
> There's no ZEND_IS_GREATER opcode, so it gets quietly turned into:
>
> if (123 < $gmp) { ... }
>
> Which will be confusing.
>

Why would this be confusing? I'd agree if this happened in userland (people
could wonder why the operators are swapped), but internally we are already
dealing with this anyway. E.g. when you implement compare_objects you have
to be aware of this (to understand stuff like the return 1 trick).


> I dealt with this in operator by having the user apply a patch before
> building:
> https://github.com/php/pecl-php-operator/blob/master/compare-greater-5.1.2.diff
>
> If we're going to bake this into ZE, then we should just add
> ZEND_IS_GREATER(_OR_EQUAL) opcode.
>

Not sure this is really necessary, but I have no problem with doing that
either.

Nikita

--047d7b41408ee4d49f04dc8af3da--