wanghan-sanechips commented on PR #3053: URL: https://github.com/apache/kvrocks/pull/3053#issuecomment-3068681914
> Yeah it would be great if we have CI support for rv support. > > Also, why TBB is downgraded? Any reason for this? 1. The compilation and testing were done on the Pioneer Box SOPHON SG2042. I completely agree that adding CI support for RISC-V would be essential for maintaining its support in the long run. 2. I downgraded TBB due to build failures on RISC-V, caused by the introduction of the `-fcf-protection=full` compiler flag in TBB v2022.2.0. Both oneTBB/cmake/compilers/Clang.cmake and oneTBB/cmake/compilers/GNU.cmake add this flag for most architectures other than aarch64, Apple, or Android. RISC-V is not excluded, so the flag caused the following error during the build: `cc1plus: error: ‘-fcf-protection=full’ is not supported for this target compilation terminated due to -Wfatal-errors.` Downgrading to an earlier version of TBB (before v2022.2.0) avoids the problematic flag and allows the build to succeed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
