Skip to content

[unittests] Enable debug info when compiling unittests when debug info is enabled. #72548

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

Merged

Conversation

gottesmm
Copy link
Contributor

I noticed this while I was debugging a failure in unittest.

@gottesmm gottesmm requested a review from etcwilde March 24, 2024 07:22
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

1 similar comment
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

Copy link
Member

@etcwilde etcwilde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you grab a trace with --trace-expand? CMake should know how to set this. It'll be set up with CMAKE_<LANG>_FLAGS_DEBUG_INIT and CMAKE_<LANG>_FLAGS_RELWITHDEBINFO_INIT. Which target is this failing with and is it on all languages in that target, or is it specific to a given language?

@gottesmm
Copy link
Contributor Author

@etcwilde I don't know why this is happening TBH. I imagine it has something to do with how LLVM sets up its unit tests. I just hit it.

…o is enabled.

I noticed this while I was debugging a failure in unittest.
@gottesmm gottesmm force-pushed the pr-9ea3226d4ec4e1094426a535db22d95597f96990 branch from e1e5e6c to a114a30 Compare March 26, 2024 22:43
@gottesmm
Copy link
Contributor Author

@swift-ci smoke test

@gottesmm
Copy link
Contributor Author

Evan and I spoke offline, he is fine with the patch.

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

Successfully merging this pull request may close these issues.

2 participants