Convert LazyTypeObject to use the Bound API#3855
Conversation
davidhewitt
left a comment
There was a problem hiding this comment.
Ah, good catch! Some quick thoughts and I wonder if this will benefit from #3705 being merged, which I'll go rebase now.
13b5d96 to
32cf969
Compare
CodSpeed Performance ReportMerging #3855 will improve performances by 10.2%Comparing 🎉 Hooray!
|
| Benchmark | main |
LilyFoote:lazy-type-object-get-or-init-bound |
Change | |
|---|---|---|---|---|
| ⚡ | sequence_from_list |
300 ns | 272.2 ns | +10.2% |
32cf969 to
3ed9290
Compare
|
@davidhewitt I think this is ready to merge. |
davidhewitt
left a comment
There was a problem hiding this comment.
Yes, indeed, thanks!
(Sorry, I opened this PR a few times but family then pulled me sideways, even though it was so short 🙈)
I found this during #3853.