Skip to content

Add os_abi arch for aie4 arch#201

Closed
donwalkarsoham wants to merge 1 commit intoXilinx:main-gefrom
donwalkarsoham:os_abi
Closed

Add os_abi arch for aie4 arch#201
donwalkarsoham wants to merge 1 commit intoXilinx:main-gefrom
donwalkarsoham:os_abi

Conversation

@donwalkarsoham
Copy link
Collaborator

Problem solved by the commit

Add marker in ELF per AIE4 architecture

Bug / issue (if any) fixed, which PR introduced the bug, how it was discovered

N/A

How problem was solved, alternative solutions (if any) and why they were rejected

1> Added OS ABI version
Note - platform team review pending on differences in aie4 architecture - Do not review

Risks (if any) associated the changes in the commit

What has been tested and how, request additional testing if necessary

Documentation impact (if any)

Signed-off-by: Donwalkar <sdonwalk@amd.com>
@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@donwalkarsoham donwalkarsoham changed the title Add os_abi arch for aie4 arch **Do not review** Add os_abi arch for aie4 arch Dec 23, 2025
Copy link
Collaborator

@larry9523 larry9523 left a comment

Choose a reason for hiding this comment

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

To make it compatible, we need to bump up ELF version too. Also, please make sure the value here matches the spec page.

@donwalkarsoham donwalkarsoham changed the title **Do not review** Add os_abi arch for aie4 arch Add os_abi arch for aie4 arch Jan 20, 2026
@donwalkarsoham
Copy link
Collaborator Author

Closing this as the new committed changes are tracked under - #217

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants