« first day (3546 days earlier)      last day (1388 days later) » 
00:00 - 21:0021:00 - 00:00

9:01 PM
Does feature freeze for PHP mean all features have to be merged or all features need to be decided on (as in no new RFCs)?
 
@IluTov merged
 
@beberlei I see, thanks!
 
the branch gets created at that point, and only bugfixes allowed from there on
 
@beberlei Good to know. I need to hurry a little with the JIT support for match and nullsafe then.
 
@IluTov ? Match is passing the vote, so any problems with it and JIT would be considered bugs.
 
9:07 PM
@Danack JIT is not supported at the moment. There are other instructions that aren't supported, in that case the engine just falls back to using the VM.
 
!!rfcs
 
oh, same ones
 
But that basically means if you're using match in a function the entire function can't be JITted, which is definitely sub optimal.
 
I don't think the JIT optimizer works that way atm, does it? I have only seen it replace specific opcodes.
 
9:10 PM
@LeviMorrison I might very well be wrong, I've only looked at it briefly when I had a JIT related issue.
If that were the case no JIT support wouldn't be so bad I guess
 
@IluTov think possibly there was misunderstanding. The match RFc is going to be passed. It doesn't need to be completed before the branching....but it would be nice. All RFCs that have been passed would be merged unless there was severe problems found.
 
@Danack Sure. But it will have to be feature complete and merged by August 4th (as that's the feature freeze)
@LeviMorrison Looking at zend_jit it does look like any failure to JITting a specific opcode of an oparray will stop JITting of that function entirely.
 
cmb
@MarkR well, isn't the actual issue that one wants to distinguish between map/dictionary (string index) and array/vector (integer index)?
 
@IluTov Then I guess I've never seen it in action because it doesn't support the opcodes I used :)
 
> zend_jit.c#L1956-L3050
 
9:21 PM
@cmb When I think 'vector' I expected it to be fully packed sequential indexes, whereas with the PHP array I don't. Equally for arrays I would expect it to either use the exact key I give it if its valid, or error out.
 
concern.jpg
 
@LeviMorrison Maybe ^^ I have no clue how complete the JIT is.
 
Having generic vect and dict in core would be an enormous benefit, I think.
 
cmb
+1
 
@beberlei Not until 18:00 UTC-6
 
9:30 PM
@LeviMorrison +1, I read today about release on Hack they made some improvements over vec and dict also
Although I don't use it much consciously, but definitely would start to use more often
Anyone seen screenshoot with @@Attribute ofrom Alex github.com/koolkode/php-src/pull/5
Sorry for repeating myself but my eyes are bleeding
 
@@ was always the worst choice :(
 
@Girgias Haha, I almost did that in my patch, but finally reverted that change before pushing. :) But I'll do it after the current one is merged. :)
 
I like how @@ looks. To me the #[] syntax ends up harder to read when the attribute takes an array argument.
 
I think no matter how much you try it's not possible to adjust font/colour etc. of @@ to leave a smaller footprint than #[
#[ is just more elegant you cannot argue with it. @@ wins only because ppl have hope we can reduce one @ sooner or later, but it also can be later than later
 
@MátéKocsis Only reason is that I'm doing dumb indexes to see all the differences compared to string offset behaviour in regards to numeric strings
I hate @@ the most too :(
I liked <<>> the most tho, but I'm in the minority there
 
9:37 PM
I like @@ the most :) So I'm a happy voter
 
Oh the vote ends Today :/
:(
 
I've just run my DI container benchmark on PHP 8. The results didn't improve significantly compared to PHP 7.4.
 
With / without JIT?
 
First, without JIT. But afterwards I also run the bench with JIT... And I saw a slowdown :(
compared to PHP 8 non-JIT
 
@brzuchal I'm not sure either one is more elegant - that's rather subjective. For me it's about balancing conciseness, familiarity, and minimal BC break.
 
9:40 PM
I need to make a write-up about my PHP code I used for my math project
And run it under JIT, as that will probably yield enourmous gains
as it's a bloody O(n^4) numerical algorithm
 
Looking forward to reading about it! :)
 
@TheodoreBrown it's almost the same BC break, but whatever it looks like we'll have to live with @@ for another 10yrs :(
 
And don't forget if we eventually make it validate at encounter, but have to account for errors or undefined, then it will be @@@MyClass
 
@TheodoreBrown I simply compare them on a screen and can see that @@ pays much more attention than #[ and it was designed to be a metadata rather than something what hurt eyes
@MarkR Oh, great I already love it
$myFunc = @@@Attribute(@@@JIT(), @@@SoManyAts()) function foo() {};
No matter what syntax we choose I love atatattributes
 
Stutteributes.
But yes, the feature itself is absolutely going to be fantastic.
 
9:47 PM
@MarkR Error suppression is not the right way to account for errors or undefined.
 
@TheodoreBrown Optional ones it would make sense.
 
Oh, I've just noticed that there some tests where JIT is quite much faster, but for the rest, I saw a slight performance decrease.
 
Did you run the JIT all code option?
 
@MarkR As I replied on list, I don't think it makes sense since whether or not there is a class for the attribute is an internal implementation detail for libraries. It shouldn't affect the syntax that consumers write.
 
As I would expect perf decrease at it needs to JIT it
But for hot paths where the engine decides to JIT it I expect it to become faster, question is how smart the engine is
 
9:49 PM
@TheodoreBrown Shouldn't attribute syntax vote be closed?
 
@TheodoreBrown Just wait until they inevitably become validated at encounter :-)
 
@NikiC Soon
 
> Not until 18:00 UTC-6
 
Eh, y'all pedantic
 
I would have closed it tbh
 
9:50 PM
I just close RFCs on the right day ... if I don't forget about them
 
Still better than me closing an RFC 4 years after the vote ended lol
 
@Girgias :rolling_on_the_floor_laughting:
 
Bleh, this PHP 8 instrumentation API just doesn't want to co-operate with me >.<
A classic case of the last 10% taking 90% of the time...
 
@MátéKocsis Are you running CLI or FPM?
 
10:05 PM
@IluTov It's FPM. And I'm using an AWS c5.large instance.
 
@MátéKocsis Ok, I thought maybe you were using CLI with no file cache which would explain a decrease as you'd be running the JIT for every execution but that wouldn't be the case here...
It'd be interesting to see what percentage of the code is actually jitted
 
No-no :) It's a battle tested config, which I have been occassionally running for years
 
Is that visible in opcache_get_status?
 
I'll have a look at that!
Unfortunately, getting percentages is a problematic thing. This benchmark can compare the performance of different DI Containers relatively well, but it's not very usable for analyzing performance changes over time
 
@MátéKocsis Not sure I follow. What I mean was that maybe loading all classes into memory and then calling opcache_get_status might give you some information on JIT and how much of those functions were actually jitted.
I'm assuming if little of your code can be jitted you'll get the overhead but little speedups as you'll just fall back on the VM. I wouldn't think code that does get jitted is actually slower than execution in the VM (but that is just a wild assumption)
 
10:22 PM
@LeviMorrison :-/
@TheodoreBrown anyhoo, congrats on the vote I guess, going to bed now, so can't see it happening ;-)
 
@beberlei Thanks! Have a good night.
 
10:45 PM
@IluTov Ah, I can't read that late ^^ I thought that you'd want to see what the difference is (in percentage) between JIT vs non-JIT :D
 
11:02 PM
I looked at the output, doesn't contain interesting information about the JIT...
Would be interesting though. Then we could look at some large code bases (symfony, laravel, etc.) to get a feeling how much of it can make use of the JIT.
 
@IluTov ..... approximately none? unless the code is doing math heavy stuff there is such a small chance of any improvement with JIT.
 
@Danack What I mean is how much of the code base can be compiled to actual machine code, not whether that will run faster or not.
 
How much of that is IO bounded?
if any
 
11:48 PM
Good evening everyone. How can I do a grouping of values by ID_USER and the values in the ID_MEDALS column, display separated by commas.
 
You select by id_user and on the result you do an implode in PHP. SQL can't do that for you
 
Unless you're using mysql, then GROUP_CONCAT
 
group_concat might be what you want. Though I would recommend doing it in PHP anyway.
 
@Derick I'm about to close the poll. Can you double-check my results? One invalid ballot not counted, quota 31, @@ elected with 34 votes.
 
SELECT id, id_user, GROUP_CONCAT(id_medals) FROM medals
Was it something like that? I believe that I am not doing it right, because the values are grouping wrong. lol
or GROUP_CONCAT(id_user)?
 
11:57 PM
@Tiago search for how to do 'group by'. you need to tell it what you want it to do.
 
SELECT id, id_user, GROUP_CONCAT(id_medals, ',') GROUP BY id_medals
 
00:00 - 21:0021:00 - 00:00

« first day (3546 days earlier)      last day (1388 days later) »