Skip to content

[Relay] make "ToScalar" support directly obtaining "int64_t"#16324

Merged
masahi merged 1 commit intoapache:mainfrom
mawnja:main
Jan 10, 2024
Merged

[Relay] make "ToScalar" support directly obtaining "int64_t"#16324
masahi merged 1 commit intoapache:mainfrom
mawnja:main

Conversation

@mawnja
Copy link
Copy Markdown
Contributor

@mawnja mawnja commented Jan 1, 2024

Because on Windows, "long double" is 64 bits instead of 128 bits like on Linux, to avoid overflow from "long double" to "int64_t"

Because on Windows, "long double" is 64 bits instead of 128 bits like on Linux, to avoid overflow from "long double" to "int64_t"
Copy link
Copy Markdown
Contributor Author

@mawnja mawnja left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Hzfengsy All checks passed, can you help me merge the changes in?

@mawnja mawnja marked this pull request as draft January 5, 2024 01:58
@mawnja mawnja marked this pull request as ready for review January 5, 2024 01:59
@mawnja
Copy link
Copy Markdown
Contributor Author

mawnja commented Jan 10, 2024

@masahi All checks passed, can you help me merge the changes in?

@masahi masahi merged commit 0b2358c into apache:main Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants