Skip to content

support native int64 as valid datatype #2937

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

Closed
wants to merge 1 commit into from
Closed

Conversation

apbose
Copy link
Collaborator

@apbose apbose commented Jun 18, 2024

No description provided.

@github-actions github-actions bot added the component: core Issues re: The core compiler label Jun 18, 2024
@github-actions github-actions bot requested a review from narendasan June 18, 2024 22:33
@HolyWu
Copy link
Contributor

HolyWu commented Jun 26, 2024

@apbose
Copy link
Collaborator Author

apbose commented Jun 26, 2024

Thanks @HolyWu for pointing out the above. @narendasan would the changes in PR https://github.com/pytorch/TensorRT/pull/2855/files#diff-97c67f0cea25ec7aa88ca61594d9ad62e9fb575cab3647b079cb718f81308454 work or would we have to create a type map for kINT32 to kINT64 in the torchscript path?

@narendasan
Copy link
Collaborator

Yeah neither of these PRs implements the feature fully since we need to either amend truncation in TorchScript or have a TorchScript specific type map

@narendasan
Copy link
Collaborator

@apbose going to close this. reopen a new pr with the comprehensive changes if its still needed

@narendasan narendasan closed this Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants