gh-131798: JIT: optimize _LOAD_COMMON_CONSTANT#146104
gh-131798: JIT: optimize _LOAD_COMMON_CONSTANT#146104trdthg wants to merge 1 commit intopython:mainfrom
Conversation
|
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
Fidget-Spinner
left a comment
There was a problem hiding this comment.
Thanks, this LGTM! Please remove the news entry, we don't need it as it's not a user facing code.
068d2bb to
19fc90f
Compare
|
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
|
_LOAD_COMMON_CONSTANT x86_64 dissasembly in I think this is prior to our CFG optimizations in the backend, so it's not 100% accurate, but pretty close. _LOAD_CONST_INLINE_BORROW: This is a really nice reduction. Good job! |
19fc90f to
b37ec56
Compare
|
We generally don't force push, because all PRs in CPython are squash-merged. So just push normal commits, thanks! |
Try to learn and make a small contribution :D