Skip to content
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

Eigen asserts always enabled #95

Open
alaurenzi opened this issue Dec 1, 2017 · 3 comments
Open

Eigen asserts always enabled #95

alaurenzi opened this issue Dec 1, 2017 · 3 comments
Assignees

Comments

@alaurenzi
Copy link
Contributor

I do not think it will hurt performance, and it will be a lot easier to debug tasks/constraints.

@EnricoMingo
Copy link
Contributor

From the online documentation we saw that is not possible to add any flag in the CMake. Any other idea?

@alaurenzi
Copy link
Contributor Author

Possible solution: custom flags to avoid the definition of NDEBUG (standard c macro that disables all asserts)

# Set a default build type for single-configuration
# CMake generators if no build type is set.
IF(NOT CMAKE_CONFIGURATION_TYPES AND NOT CMAKE_BUILD_TYPE)
   SET(CMAKE_BUILD_TYPE RelWithDebInfo)
ENDIF(NOT CMAKE_CONFIGURATION_TYPES AND NOT CMAKE_BUILD_TYPE)

# Select flags.
SET(CMAKE_CXX_FLAGS_RELWITHDEBINFO "-O2 -g")
SET(CMAKE_CXX_FLAGS_RELEASE "-O3")
SET(CMAKE_CXX_FLAGS_DEBUG  "-O0 -g")

@EnricoMingo
Copy link
Contributor

I'll try it in the fb_estimation branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants