Skip to content

ROCm: Remove -Wno-interference-size compiler flag - #24326

Merged
snnn merged 2 commits into
microsoft:mainfrom
jagadish-amd:fix_interference-size_error
Apr 8, 2025
Merged

ROCm: Remove -Wno-interference-size compiler flag#24326
snnn merged 2 commits into
microsoft:mainfrom
jagadish-amd:fix_interference-size_error

Conversation

@jagadish-amd

Copy link
Copy Markdown
Contributor

hipClang does not support -Wno-interference-size.
Hence remove the option to avoid build error.

hipClang does not support -Wno-interference-size.
Hence remove the option to avoid build error.

Signed-off-by: Jagadish Krishnamoorthy <jagadish.krishnamoorthy@amd.com>
@jagadish-amd

Copy link
Copy Markdown
Contributor Author

ping @jeffdaily @causten

@tianleiwu

Copy link
Copy Markdown
Contributor

/azp run Windows ARM64 QNN CI Pipeline,Windows x64 QNN CI Pipeline,Windows GPU Doc Gen CI Pipeline,ONNX Runtime Web CI Pipeline,Win_TRT_Minimal_CUDA_Test_CI,Linux CPU CI Pipeline,Linux CPU Minimal Build E2E CI Pipeline,Linux GPU CI Pipeline,Linux GPU TensorRT CI Pipeline,Linux OpenVINO CI Pipeline

@tianleiwu

Copy link
Copy Markdown
Contributor

/azp run Linux QNN CI Pipeline,onnxruntime-binary-size-checks-ci-pipeline,Big Models,Linux Android Emulator QNN CI Pipeline,Android CI Pipeline,iOS CI Pipeline,ONNX Runtime React Native CI Pipeline,Linux DNNL CI Pipeline,Linux MIGraphX CI Pipeline,Linux ROCm CI Pipeline

@azure-pipelines

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

@azure-pipelines

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

@tianleiwu

Copy link
Copy Markdown
Contributor

@jagadish-amd, could you merge latest main to fix build pipelines?

@tianleiwu

Copy link
Copy Markdown
Contributor

/azp run Windows ARM64 QNN CI Pipeline,Windows x64 QNN CI Pipeline,Windows GPU Doc Gen CI Pipeline,ONNX Runtime Web CI Pipeline,Win_TRT_Minimal_CUDA_Test_CI,Linux CPU CI Pipeline,Linux CPU Minimal Build E2E CI Pipeline,Linux GPU CI Pipeline,Linux GPU TensorRT CI Pipeline,Linux OpenVINO CI Pipeline

@tianleiwu

Copy link
Copy Markdown
Contributor

/azp run Linux QNN CI Pipeline,onnxruntime-binary-size-checks-ci-pipeline,Big Models,Linux Android Emulator QNN CI Pipeline,Android CI Pipeline,iOS CI Pipeline,ONNX Runtime React Native CI Pipeline,Linux DNNL CI Pipeline,Linux MIGraphX CI Pipeline,Linux ROCm CI Pipeline

@azure-pipelines

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

@azure-pipelines

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

@snnn

snnn commented Apr 8, 2025

Copy link
Copy Markdown
Contributor

Will merge it first. Due to some limitations in our CI system, we cannot test this change.

@snnn
snnn merged commit d7a38a5 into microsoft:main Apr 8, 2025
zhaoxul-qti pushed a commit to CodeLinaro/onnxruntime that referenced this pull request Apr 17, 2025
hipClang does not support -Wno-interference-size.
Hence remove the option to avoid build error.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants