-
Notifications
You must be signed in to change notification settings - Fork 749
Implement i64.div and i64.rem intrinsics #1375
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
Conversation
VELAPLATFO-2379 Signed-off-by: Huang Qi <huangqi3@xiaomi.com> Change-Id: Ia26a11e89d6967ecef104f2e46658746580f6ad7
VELAPLATFO-2379 Signed-off-by: Huang Qi <huangqi3@xiaomi.com> Change-Id: Ie6eb2a586d351ec6ca06af9bc1e4f8c3eb7bf846
Change-Id: I695507871cf84472636a114c7ee8cf20734ecb07
| } | ||
|
|
||
| static void | ||
| add_i64_common_intrinsics(AOTCompContext *comp_ctx) |
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.
who is going to call this function?
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.
I guess all 32-bit platform will need it, but only riscv32 verified now.
lum1n0us
left a comment
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.
Just one question
|
LGTM |
No description provided.