Skip to content

Conversation

@djeong20
Copy link
Contributor

@djeong20 djeong20 commented Dec 5, 2025

This PR addresses known Meson limitations in applying b_vscrt settings for clang++.exe in C++ builds.
This ensures debug behavior matches Visual Studio Debug configurations when using buildtype=debug.

Self-evaluation:

  1. Build test: [X]Passed [ ]Failed [ ]Skipped
  2. Run test: [X]Passed [ ]Failed [ ]Skipped

This PR addresses known Meson limitations in applying b_vscrt settings for clang++.exe in C++ builds.
This ensures debug behavior matches Visual Studio Debug configurations when using buildtype=debug.

**Self-evaluation:**
1. Build test: [X]Passed [ ]Failed [ ]Skipped
2. Run test:   [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Donghyeon Jeong <[email protected]>
Copy link
Collaborator

@jijoongmoon jijoongmoon left a comment

Choose a reason for hiding this comment

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

LGTM

# Just setting buildtype=debug may not switch the CRT or iterator level.
# Therefore, forcing _ITERATOR_DEBUG_LEVEL=2 for Meson debug build to match VS Debug
if build_machine.system() == 'windows'
add_project_arguments('-D_ITERATOR_DEBUG_LEVEL=2', language : 'cpp')
Copy link
Contributor

@songgot songgot Dec 5, 2025

Choose a reason for hiding this comment

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

:)

Suggested change
add_project_arguments('-D_ITERATOR_DEBUG_LEVEL=2', language : 'cpp')
add_project_arguments('-D_ITERATOR_DEBUG_LEVEL=2', language: 'cpp')

@djeong20 djeong20 marked this pull request as draft December 5, 2025 02:19
Copy link
Contributor

@songgot songgot left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants