-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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.
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
Change _setindex implementation for tuples to improve constant propagation #40635
Conversation
Update fork
Change _setindex to improve constant propagation
Would be good to add a test. |
Thank you for the feedback. I added an |
The failure of |
I would like to "revive" this PR; @KristofferC can you provide some feedback on this? Let me give some context again: the current @mbauman perhaps you could review this PR since it is a faithful follow-up of your suggestion on Discourse (cf. https://discourse.julialang.org/t/selectdim-behaviour-for-arrays-of-dimension-greater-than-2/52211)? Best regards Note: the failure of |
Could you try rebase this on the master branch to perhaps fix the windows CI? When CI is green, I think this can be merged. Feel free to keep pinging if it gets forgotten again. |
@KristofferC thank you for the feedback. I updated the PR in accordance with your review. All tests have passed. |
Friendly ping about the prospective merging of this PR with the master branch as suggested by @KristofferC 😊 |
Resolves #39944