Skip to content

[QNN-EP] Translate FP-to-Bool Cast by NotEqual. #24466

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

minfhong-quic
Copy link
Contributor

Description

For QNN-EP, build FP-to-Bool Cast into NotEqual.

Motivation and Context

HTP currently does not support FP-to-Bool Cast due to some limitations. To unblock CLIP models, replace such Cast with NotEqual to achieve the same functionality.

@minfhong-quic
Copy link
Contributor Author

@microsoft-github-policy-service agree company="Qualcomm"

@HectorSVC HectorSVC added the ep:QNN issues related to QNN exeution provider label Apr 18, 2025
@HectorSVC
Copy link
Contributor

/azp run Linux QNN CI Pipeline,Windows x64 QNN CI Pipeline,Windows ARM64 QNN CI Pipeline,Linux Android Emulator QNN CI Pipeline

Copy link

Azure Pipelines successfully started running 4 pipeline(s).

@HectorSVC
Copy link
Contributor

please fix the Windows build error and code format issue

HTP currently does not support FP-to-Bool Cast due to some limitations.
To unblock CLIP models, replace such Cast with NotEqual to achieve the
same functionality.
Test: Add UT testcase for FP/FP16 to Bool.
@minfhong-quic minfhong-quic force-pushed the dev/minfhong-quic/fpcastbool-to-notequal branch from c45fb8a to add6952 Compare April 21, 2025 08:07
@adrianlizarraga
Copy link
Contributor

/azp run Linux QNN CI Pipeline, Win_TRT_Minimal_CUDA_Test_CI, Windows ARM64 QNN CI Pipeline, Windows GPU Doc Gen CI Pipeline, Windows x64 QNN CI Pipeline

Copy link

Azure Pipelines successfully started running 5 pipeline(s).

@adrianlizarraga
Copy link
Contributor

/azp run web_Debug / build_onnxruntime_web, web_Release / build_onnxruntime_web

Copy link

No pipelines are associated with this pull request.

@adrianlizarraga
Copy link
Contributor

/azp run "web_Debug / build_onnxruntime_web", "web_Release / build_onnxruntime_web"

Copy link

No pipelines are associated with this pull request.

@adrianlizarraga adrianlizarraga merged commit c7b901f into microsoft:main Apr 21, 2025
70 checks passed
@adrianlizarraga
Copy link
Contributor

Hi @vraspar FYI, to be cherry-picked for ORT 1.22.0

adrianlizarraga added a commit that referenced this pull request Apr 22, 2025
### Description
Removes unnecessary std::move on an r-value expression. This caused a
compiler warning/error in the Linux Android QNN pipeline.



### Motivation and Context
Introduced by PR: #24466
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 release:1.22.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants