Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Invoke() using ReadOnlySpan, and allocation optimizations #1161

Merged
merged 33 commits into from
Feb 20, 2025
Merged

Invoke() using ReadOnlySpan, and allocation optimizations #1161

merged 33 commits into from
Feb 20, 2025
+1,262 −305

Conversation

jakubmisek
Copy link
Member

No description provided.

arontsang and others added 30 commits February 14, 2025 01:49
@arontsang
Span based Invoke
80c577b
@arontsang
Rename BindCore to Bind overload
87e4d4a
@arontsang
Fix OverloadBinder.cs where fromarg != 0
bfe2418
@jakubmisek
Merge branch 'master' into feature/span-invoke-2
28dbb20
@jakubmisek
cleanups, fixes compile warnings, avoids using/defining old Invoke(Co…
8e6df97 
…ntext, Array)
@jakubmisek
more use of IPhpCallable.Invoke(Context, ReadOnlySpan)
0c69688
@arontsang
Clean up old code
ab2a840
@arontsang
Extension method helpers to reduce allocation when calling IPhpCallab…
2e60ced 
…le.Invoke
@jakubmisek
resolve correct Invoke() during compile and dynamic compile
8b21fb7 
+ fixes a few compile warnings
@jakubmisek
Merge branch 'master' into feature/span-invoke-2
bae73dc
@jakubmisek
update readme, NET 9 SDK required to build platform
d023b38
@jakubmisek
build fix
0284fe4
@arontsang @jakubmisek
Add T4 template for ArrayPool overloads upto 16
0570074
@jakubmisek
Merge branch 'master' into feature/span-invoke-2
a957ccf 
# Conflicts:
#	src/Peachpie.CodeAnalysis/Symbols/CoreType.cs
#	src/Peachpie.CodeAnalysis/Symbols/TypeSymbolExtensions.cs
@jakubmisek
build: suppress CS0618
a78b6a4
@jakubmisek
prepared for use of params ReadOnlySpan<T>
ca015c3
@jakubmisek
fix OverloadBinder to use ReadOnlySpan<PhpValue>, cleanups
62417d2 
- moved MethodInfo cache to Cache
- resolve methods only once (make them static)
- WordPress runs
@jakubmisek
avoids allocs in array_filter()
c93923f
@jakubmisek
avoids Substring()
ff608c5
@jakubmisek
avoids alloc in hash() functions
c58b4db 
- avoids ToArray()
- avoids Action<> closure
@jakubmisek
avoids more allocs in MD5
63d82be
@jakubmisek
fixed unwanted alloc in BinToHex()
24efbfe
@jakubmisek
allocs in md5, fix
85b5f69
@jakubmisek
avoids alloc in unserialize()
dc744ce
@jakubmisek
use more Span
095000b
@jakubmisek
less alocs in curl
5c63471
@jakubmisek
IPhpCallable.Invoke(ctx, p0) overload without array buffer
0c1efe7
@jakubmisek
fix memory leak in Closure, avoids allocation
d9e58d4
@jakubmisek
implicit conversion PhpArray -> ReadOnlySpan<PhpValue>
5eaa3b0
@jakubmisek
pooling array internal array to lower GC pressure
b44ea49
@jakubmisek jakubmisek self-assigned this Feb 20, 2025
@jakubmisek jakubmisek merged commit 9f5564b into master Feb 20, 2025
3 of 5 checks passed
@jakubmisek jakubmisek mentioned this pull request Feb 20, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jakubmisek jakubmisek

Labels
⏫ enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jakubmisek @arontsang