-
Notifications
You must be signed in to change notification settings - Fork 497
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
error: "__always_inline" redefined [-Werror] #366
Comments
No body at home? |
@ElyesH Sorry for the delay. Most of the discussions generally happen on mailing list so Github issues get overlooked. I see you have raised a GitHub PR as well. I will send your patch to the mailing list. |
avpatel
pushed a commit
to avpatel/opensbi
that referenced
this issue
Sep 15, 2024
This is to fix update opensbi to upstream build for coreboot. https://qa.coreboot.org/job/coreboot-gerrit/257449/testReport/junit/(root)/clang/EMULATION_QEMU_RISCV_RV64_/ Closes: riscv-software-src#366 Signed-off-by: Elyes Haouas <[email protected]> Signed-off-by: Anup Patel <[email protected]>
avpatel
pushed a commit
to avpatel/opensbi
that referenced
this issue
Sep 15, 2024
Update __always_inline macro define to fix opensbi upstream build for coreboot. Refer, https://qa.coreboot.org/job/coreboot-gerrit/257449/testReport/junit/(root)/clang/EMULATION_QEMU_RISCV_RV64_/ Closes: riscv-software-src#366 Signed-off-by: Elyes Haouas <[email protected]> Signed-off-by: Anup Patel <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We can't update opensbi submodule to upstream (since 2022 !) as it fails to build :
CC ramstage/arch/riscv/misc.o CC ramstage/arch/riscv/opensbi.o In file included from 3rdparty/opensbi/include/sbi/fw_dynamic.h:46, from src/arch/riscv/opensbi.c:8: 3rdparty/opensbi/include/sbi/sbi_types.h:72:9: error: "__always_inline" redefined [-Werror] 72 | #define __always_inline inline __attribute__((always_inline)) | ^~~~~~~~~~~~~~~ In file included from <command-line>: ./src/commonlib/bsd/include/commonlib/bsd/compiler.h:58:9: note: this is the location of the previous definition 58 | #define __always_inline inline __attribute__((__always_inline__)) | ^~~~~~~~~~~~~~~ cc1: all warnings being treated as errors make[2]: *** [Makefile:423: /cb-build/coreboot-gerrit.0/gcc/EMULATION_QEMU_RISCV_RV64_/ramstage/arch/riscv/opensbi.o] Error 1 make[2]: Leaving directory '/home/coreboot/node-root/workspace/coreboot-gerrit'
(see https://qa.coreboot.org/job/coreboot-gerrit/264570/testReport/junit/(root)/gcc/EMULATION_QEMU_RISCV_RV64_/)
The text was updated successfully, but these errors were encountered: