Skip to content

[VITISAI] fix compile error for onnxruntime - #17252

Merged
jywu-mysoft merged 1 commit into
microsoft:mainfrom
BoarQing:fix_vitisai_build
Aug 23, 2023
Merged

[VITISAI] fix compile error for onnxruntime#17252
jywu-mysoft merged 1 commit into
microsoft:mainfrom
BoarQing:fix_vitisai_build

Conversation

@BoarQing

Copy link
Copy Markdown
Contributor

Description

Updated the code to pass in the missing parameter

Motivation and Context

Compile error. See #17139

@jywu-mysoft

Copy link
Copy Markdown
Contributor

/azp run Windows ARM64 QNN CI Pipeline,Windows CPU CI Pipeline,Windows GPU CI Pipeline,Windows GPU TensorRT CI Pipeline,onnxruntime-binary-size-checks-ci-pipeline,orttraining-linux-ci-pipeline,orttraining-linux-gpu-ci-pipeline,orttraining-ortmodule-distributed

@jywu-mysoft

Copy link
Copy Markdown
Contributor

/azp run Linux CPU CI Pipeline,Linux CPU Minimal Build E2E CI Pipeline,Linux GPU CI Pipeline,Linux GPU TensorRT CI Pipeline,Linux OpenVINO CI Pipeline,Linux QNN CI Pipeline,MacOS CI Pipeline,ONNX Runtime Web CI Pipeline

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 8 pipeline(s).

1 similar comment
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 8 pipeline(s).

@jywu-mysoft
jywu-mysoft merged commit d21a2f0 into microsoft:main Aug 23, 2023
@faxu faxu added the triage:approved Approved for cherrypicks for release label Aug 24, 2023
Lafi7e pushed a commit that referenced this pull request Aug 28, 2023
### Description
<!-- Describe your changes. -->
Updated the code to pass in the missing parameter


### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
Compile error. See #17139

Co-authored-by: Yueqing Zhang <yueqingz@amd.com>
snnn pushed a commit that referenced this pull request Aug 28, 2023
@BoarQing
BoarQing deleted the fix_vitisai_build branch August 31, 2023 08:45
kleiti pushed a commit to kleiti/onnxruntime that referenced this pull request Mar 22, 2024
### Description
<!-- Describe your changes. -->
Updated the code to pass in the missing parameter


### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->
Compile error. See microsoft#17139

Co-authored-by: Yueqing Zhang <yueqingz@amd.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

triage:approved Approved for cherrypicks for release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants