Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

i#3544 RV64: Added machine type support for ELF hdr verification #6088

Merged
merged 1 commit into from
May 30, 2023

Conversation

ksco
Copy link
Contributor

@ksco ksco commented May 27, 2023

This patch added ELF header checks for RISC-V.

Issue: #3544

@ksco ksco requested a review from abhinav92003 May 27, 2023 08:15
@ksco ksco added this pull request to the merge queue May 30, 2023
Merged via the queue into DynamoRIO:master with commit 2597fa5 May 30, 2023
15 checks passed
@ksco ksco deleted the elf branch May 30, 2023 20:56
@Reese0710
Copy link

I don't know if you ran into this problem when I tried to compile the latest version of the dynamorio like this
image
then it's failed
image
I've tried the gcc 7.3.0 & 8.5.0
my OS is centos 8.0 x86, cpu: Intel(R) Xeon(R) Gold 6278 CPU @ 2.60GHz

@ksco
Copy link
Contributor Author

ksco commented Jul 12, 2023

RISC-V is kinda new, upgrading the compiler to a newer version should fix this. I don't know if we're still supporting old compilers, can you open a new issue so we can discuss it there?

@derekbruening
Copy link
Contributor

Generally if people want support for old or different toolchains that are not part of existing Github Actions continuous testing (Ubuntu 20 and 22 for Linux) we ask that those who want that support do the work of adding and maintaining it as it is not reasonable for the small team of regular contributors to support every toolchain out there.

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.

None yet

4 participants