-
Notifications
You must be signed in to change notification settings - Fork 58
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
Cannot compile with Profile build type #158
Comments
I don't have that issue, but a different one. Specifically stf is built with LTO while the linking stage doesn't accept it. Got a flag mismatch. Have you tried with |
Same error as this?
|
Managed to fix it by moving the setup_stf_linker out of the Release build only
Will create PR |
Yep. That’s the one. Thanks for debugging On Mar 5, 2024, at 4:42 PM, danbone ***@***.***> wrote:
Managed to fix it by moving the setup_stf_linker out of the Release build only
diff --git a/CMakeLists.txt b/CMakeLists.txt
index bee99e2..4fc4298 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -38,9 +38,10 @@ set (DISABLE_STF_DOXYGEN ON)
if (CMAKE_BUILD_TYPE MATCHES "^[Rr]elease")
set (FULL_LTO true)
- include(${STF_LIB_BASE}/cmake/stf_linker_setup.cmake)
- setup_stf_linker(false)
endif()
+include(${STF_LIB_BASE}/cmake/stf_linker_setup.cmake)
+setup_stf_linker(false)
+
# Use ccache if installed
find_program (CCACHE_PROGRAM ccache)
Will create PR
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: ***@***.***>
|
Fix for #158 Co-authored-by: Daniel Bone <[email protected]>
@danbone Can I take up this issue? |
Unless Dan disagrees, feel free to take it. I just reproduced the original issue. |
@klingaard didn't this get fixed by |
@danbone Can I take up this issue? |
Yes, absolutely fine by me |
I'm trying to compile with CMAKE_BUILD_TYPE=Profile but I'm getting the following error:
Has anyone else encountered this?
The text was updated successfully, but these errors were encountered: