Skip to content

Add ppc64le (PowerPC) support to onnxruntime-extensions.#1041

Merged
sayanshaw24 merged 3 commits intomicrosoft:mainfrom
BODAPATIMAHESH:main_ppc64le_port
Apr 11, 2026
Merged

Add ppc64le (PowerPC) support to onnxruntime-extensions.#1041
sayanshaw24 merged 3 commits intomicrosoft:mainfrom
BODAPATIMAHESH:main_ppc64le_port

Conversation

@BODAPATIMAHESH
Copy link
Copy Markdown
Contributor

Key updates:

  • Fix compilation failures due to missing libpng PowerPC sources
  • Replace dlib bundled libpng with upstream libpng for ppc64le
  • Handle architecture detection via CMAKE_SYSTEM_PROCESSOR
  • Built shared library and Python wheel on ppc64le
  • Used externally built ONNX Runtime via:
    • ONNXRUNTIME_INCLUDE_DIR
    • ONNXRUNTIME_LIB_DIR

Key updates:
- Fix compilation failures due to missing libpng PowerPC sources
- Replace dlib bundled libpng with upstream libpng for ppc64le
- Handle architecture detection via CMAKE_SYSTEM_PROCESSOR
- Built shared library and Python wheel on ppc64le
- Used externally built ONNX Runtime via:
  - ONNXRUNTIME_INCLUDE_DIR
  - ONNXRUNTIME_LIB_DIR
@BODAPATIMAHESH BODAPATIMAHESH requested a review from a team as a code owner March 31, 2026 12:34
@sayanshaw24
Copy link
Copy Markdown
Collaborator

sayanshaw24 commented Apr 8, 2026

@BODAPATIMAHESH Thanks for adding this! Is there a customer requirement for PowerPC/if so could you please describe what it is and why we need this?

endif()

# PowerPC VSX optimizations
if(CMAKE_SYSTEM_PROCESSOR MATCHES "ppc64|ppc64le")
Copy link
Copy Markdown
Collaborator

@sayanshaw24 sayanshaw24 Apr 8, 2026

Choose a reason for hiding this comment

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

can we have one if/else here instead of 2 if's in case the processor name is something unexpected (check for ppc and fallback to current arm srcs)?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated it. Please review @https://github.com/sayanshaw24

@BODAPATIMAHESH
Copy link
Copy Markdown
Contributor Author

@BODAPATIMAHESH Thanks for adding this! Is there a customer requirement for PowerPC/if so could you please describe what it is and why we need this?

Our customer is using ONNX Runtime for text and vision embedding generation and plans to integrate onnxruntime-extensions. To support this requirement, we ported onnxruntime-extensions to the Power platform and created a PR.

@sayanshaw24
Copy link
Copy Markdown
Collaborator

@BODAPATIMAHESH Thanks for adding this! Is there a customer requirement for PowerPC/if so could you please describe what it is and why we need this?

Our customer is using ONNX Runtime for text and vision embedding generation and plans to integrate onnxruntime-extensions. To support this requirement, we ported onnxruntime-extensions to the Power platform and created a PR.

Sounds good, what company or organization are you yourself affiliated with? We'd like to track usage, so this does not go stale in the future.

@BODAPATIMAHESH
Copy link
Copy Markdown
Contributor Author

@BODAPATIMAHESH Thanks for adding this! Is there a customer requirement for PowerPC/if so could you please describe what it is and why we need this?

Our customer is using ONNX Runtime for text and vision embedding generation and plans to integrate onnxruntime-extensions. To support this requirement, we ported onnxruntime-extensions to the Power platform and created a PR.

Sounds good, what company or organization are you yourself affiliated with? We'd like to track usage, so this does not go stale in the future.

I’m affiliated with IBM, and we’re using onnxruntime-extensions with ONNX Runtime for text and vision embedding generation on Power architecture.

@sayanshaw24 sayanshaw24 enabled auto-merge (squash) April 10, 2026 23:43
@sayanshaw24 sayanshaw24 merged commit 3ee90f4 into microsoft:main Apr 11, 2026
32 checks passed
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.

2 participants