Skip to content
New issue

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

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

Already on GitHub? # to your account

Fix softfloat build failure #266

Merged
merged 2 commits into from
Nov 16, 2023

Conversation

visitorckw
Copy link
Collaborator

Addressed compilation errors on macOS/Arm64 arising from conflicts between SoftFloat and SDL headers. The solution involves explicitly defining the conflicting types in SoftFloat to avoid clashes.

Close: #261

#undef float16_t
#undef bfloat16_t
#undef float32_t
#undef float64_t

Check notice

Code scanning / Cppcheck (reported by Codacy)

MISRA 20.5 rule

MISRA 20.5 rule
#include "softfloat/softfloat.h"
#undef float16_t

Check notice

Code scanning / Cppcheck (reported by Codacy)

MISRA 20.5 rule

MISRA 20.5 rule
#include "softfloat/softfloat.h"
#undef float16_t
#undef bfloat16_t
#undef float32_t

Check notice

Code scanning / Cppcheck (reported by Codacy)

MISRA 20.5 rule

MISRA 20.5 rule
#include "softfloat/softfloat.h"
#undef float16_t
#undef bfloat16_t

Check notice

Code scanning / Cppcheck (reported by Codacy)

MISRA 20.5 rule

MISRA 20.5 rule
Replace the usage of the softfloat type 'float32_t' with the custom
'riscv_float_t' type defined within rv32emu. This modification ensures
better integration and consistency within the rv32emu codebase.
Addressed compilation errors on macOS/Arm64 arising from conflicts
between SoftFloat and SDL headers. The solution involves explicitly
defining the conflicting types in SoftFloat to avoid clashes.
@visitorckw visitorckw force-pushed the fix-softfloat-build-failure branch from 42c10f6 to 129aca2 Compare November 16, 2023 19:05
@jserv jserv merged commit dcf217d into sysprog21:master Nov 16, 2023
@visitorckw visitorckw deleted the fix-softfloat-build-failure branch November 16, 2023 19:48
vestata pushed a commit to vestata/rv32emu that referenced this pull request Jan 24, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Build failure with recent SoftFloat integration on macOS/Arm64
2 participants