Skip to content

Fix the issue for Gather int64 indices handling - #23274

Merged
HectorSVC merged 4 commits into
mainfrom
qnn_gather_int64_indicies
Jan 8, 2025
Merged

Fix the issue for Gather int64 indices handling#23274
HectorSVC merged 4 commits into
mainfrom
qnn_gather_int64_indicies

Conversation

@HectorSVC

Copy link
Copy Markdown
Contributor

Description

Fix the issue for Gather int64 indices handling. Make it still insert Cast node if it's non-quantized Gather node.

@HectorSVC HectorSVC added the ep:QNN issues related to QNN exeution provider label Jan 7, 2025

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You can commit the suggested changes from lintrunner.

Comment thread onnxruntime/core/providers/qnn/builder/qnn_model_wrapper.h
HectorSVC and others added 2 commits January 7, 2025 09:24
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@HectorSVC
HectorSVC merged commit 76d6345 into main Jan 8, 2025
@HectorSVC
HectorSVC deleted the qnn_gather_int64_indicies branch January 8, 2025 20:52
snnn pushed a commit that referenced this pull request Jan 8, 2025
### Description
Fix the issue for Gather int64 indices handling. Make it still insert Cast node if it's non-quantized Gather node.
tarekziade pushed a commit to tarekziade/onnxruntime that referenced this pull request Jan 10, 2025
### Description
Fix the issue for Gather int64 indices handling. Make it still insert Cast node if it's non-quantized Gather node.
guschmue pushed a commit that referenced this pull request Jan 12, 2025
### Description
Fix the issue for Gather int64 indices handling. Make it still insert Cast node if it's non-quantized Gather node.
ashrit-ms pushed a commit that referenced this pull request Mar 17, 2025
### Description
Fix the issue for Gather int64 indices handling. Make it still insert Cast node if it's non-quantized Gather node.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ep:QNN issues related to QNN exeution provider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants