-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Add regression test for broken non commutative multiplication (#9175) #9218
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
base: master
Are you sure you want to change the base?
Conversation
Current test output is:
|
336e6e2
to
f4a123b
Compare
Okay I've got a fix, which is to remove the Not sure how much this will affect performances, and or if there is another way to handle it. |
As it might not be when an internal object overloads the multiplication operation
f4a123b
to
e0350f4
Compare
This shouldn't affect performance directly. This will slightly increase VM size and reduce code locality. At first, we allow to break operators commutativity, then we will need a user defined operators order, etc |
This should add a test to catch the issue pointed out by #9175
Currently XFAIL as well it is broken.
Not sure this is the best way to write an internal object so reviews on this aspect would be appreciated.