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.

Sign up for GitHub

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

Jump to bottom

fixed missing cstdint includes + fixed narrowing conversion error on RISC-V platform #13

Merged

gaspardpetit merged 2 commits into gaspardpetit:main from vogma:fix_compile_errors

Oct 15, 2024

Contributor

vogma commented Sep 30, 2024

This pull request includes two small changes.

On all the machines i tested on (2x x86 and 1x RISC-V), i got errors about missing cstdint includes in the project when using the gcc compiler. I added the missing includes to the header files.
When compiling the benchmark-suite on a RISC-V architecture, a narrowing conversion error occurs. This PR adds an explicit signed to the corresponding char array.

vogma added 2 commits

September 30, 2024 16:45


          fixed missing cstdint includes

0b642b6


          fixed narrowing conversion error on RISC-V platform

729cb3b

gaspardpetit merged commit 143c4a2 into gaspardpetit:main

Owner

gaspardpetit commented Oct 15, 2024

Thank you!

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

Labels

None yet