Skip to content

Add dynamic dispatch to AIX and clang for Power#4280

Merged
martin-frbg merged 28 commits into
OpenMathLib:developfrom
ChipKerchner:dynamicDispatchAIXandClang
Nov 2, 2023
Merged

Add dynamic dispatch to AIX and clang for Power#4280
martin-frbg merged 28 commits into
OpenMathLib:developfrom
ChipKerchner:dynamicDispatchAIXandClang

Conversation

@ChipKerchner

@ChipKerchner ChipKerchner commented Nov 1, 2023

Copy link
Copy Markdown
Contributor

Known issue (to be done as a separate PR): compiling with gcc/xlf in AIX.
Works with Linux - gcc & clang, AIX - gcc/gfortran, xlc/xlf.

This adds dynamic dispatch to AIX. It also adds dynamic dispatch for clang/LLVM for Linux. All for PowerPC

Comment thread driver/others/dynamic_power.c Outdated
#define CPU_UNKNOWN 0
#define CPU_POWER5 5
#define CPU_POWER6 6
#define CPU_POWER7 7

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

POWER7 currently does not exist as an actual, unique target - it is an alias for POWER6. Do you plan to change that ?

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.

Fixed

@martin-frbg martin-frbg left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

thanks - I guess arch==power7 would still be valid and return CPU_POWER6 though?

@ChipKerchner

Copy link
Copy Markdown
Contributor Author

Fixed. We would like these changes in the next release (Nov 5th)

@martin-frbg martin-frbg added this to the 0.3.25 milestone Nov 2, 2023
@martin-frbg
martin-frbg merged commit 5944441 into OpenMathLib:develop Nov 2, 2023
@ChipKerchner
ChipKerchner deleted the dynamicDispatchAIXandClang branch August 1, 2024 20:01
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