-
-
Notifications
You must be signed in to change notification settings - Fork 40
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
(Do not merge yet) Add simple float.max specialization #4811
base: branch/default
Are you sure you want to change the base?
Conversation
I don't know a place to put this
Not sure how to run it yet, though
Figure out how to run the runner tests (they are parameterized): ./pytest.py -vv rpython/jit/backend/x86/test/test_runner.py
Turns out I was accidentally passing in the address of the function instead of the args.
In GitLab by @tekknolagi on Sep 28, 2023, 19:06 requested review from @cfbolz |
In GitLab by @tekknolagi on Sep 28, 2023, 19:07 I want to add some test for disassembly |
In GitLab by @tekknolagi on Sep 28, 2023, 19:15 added 1 commit
|
In GitLab by @tekknolagi on Sep 28, 2023, 19:19 added 1 commit
|
In GitLab by @tekknolagi on Sep 28, 2023, 19:28 added 1 commit
|
In GitLab by @tekknolagi on Oct 2, 2023, 04:05 added 9 commits
|
return max(a, b) | ||
res = self.interp_operations(f, [-5.25, 3]) | ||
assert res == 3 | ||
# TODO(emacs): Figure out why nan causes issues here |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be because "If only one value is a NaN (SNaN or QNaN) for this instruction, the second source operand, either a NaN or a valid floating-point value, is written to the result"?
In GitLab by @tekknolagi on Sep 28, 2023, 19:06
--HG--
branch : mb-dup