Skip to content

Fix Reduced Op build with empty FP16 kernel function tables#21038

Merged
adrianlizarraga merged 1 commit into
mainfrom
adrianl/reduced-op-empty-fp16-kernel-func-table
Jun 14, 2024
Merged

Fix Reduced Op build with empty FP16 kernel function tables#21038
adrianlizarraga merged 1 commit into
mainfrom
adrianl/reduced-op-empty-fp16-kernel-func-table

Conversation

@adrianlizarraga
Copy link
Copy Markdown
Contributor

@adrianlizarraga adrianlizarraga commented Jun 13, 2024

Description

  • Fixes compilation error for "reduced operator" builds with no FP16 kernels and MLAS_F16VEC_INTRINSICS_SUPPORTED enabled.
  • Fixes linker error for "reduced operator" builds with QNN EP by excluding QNN EP unit tests. QNN EP unit tests require CPU EP operator implementations to evaluate accuracy.

Motivation and Context

Need to be able to build a reduced operator build with QNN EP. See https://github.com/microsoft/onnxruntime/blob/main/docs/Reduced_Operator_Kernel_build.md

The following example operator config file causes a compilation error when either MLAS_F16VEC_INTRINSICS_SUPPORTED is defined or QNN EP is enabled.

# reduced_op_config.txt
ai.onnx;12;Add
python tools\ci_build\build.py --include_ops_by_config reduced_op_config.txt --config Debug --build_wheel --build_shared_lib --skip_tests --build_dir build --parallel --use_qnn --qnn_home '<QNN_ROOT_DIR>'

@adrianlizarraga adrianlizarraga added the ep:QNN issues related to QNN exeution provider label Jun 13, 2024
@adrianlizarraga adrianlizarraga marked this pull request as ready for review June 13, 2024 23:04
@adrianlizarraga adrianlizarraga requested a review from edgchen1 June 14, 2024 20:05
@adrianlizarraga adrianlizarraga merged commit 8f0e896 into main Jun 14, 2024
@adrianlizarraga adrianlizarraga deleted the adrianl/reduced-op-empty-fp16-kernel-func-table branch June 14, 2024 21:23
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.

3 participants