Skip to content

Fix incorrect CMAKE define SLANG_ENABLE_IR_BREAK_ALLOC,#7418

Merged
ArielG-NV merged 3 commits intoshader-slang:masterfrom
ArielG-NV:fix-cmake-issue
Jun 12, 2025
Merged

Fix incorrect CMAKE define SLANG_ENABLE_IR_BREAK_ALLOC,#7418
ArielG-NV merged 3 commits intoshader-slang:masterfrom
ArielG-NV:fix-cmake-issue

Conversation

@ArielG-NV
Copy link
Copy Markdown
Contributor

fixes: #7417
Define was called SLANG_ENABLE_IR_BREAK_ALLOC,. This is incorrect, supposed to be SLANG_ENABLE_IR_BREAK_ALLOC. Without this change, this define does nothing.

fixes: shader-slang#7417
Define was called `SLANG_ENABLE_IR_BREAK_ALLOC,`. This is incorrect, supposed to be `SLANG_ENABLE_IR_BREAK_ALLOC`.
@ArielG-NV ArielG-NV requested a review from a team as a code owner June 11, 2025 19:59
@ArielG-NV ArielG-NV changed the title fix SLANG_ENABLE_IR_BREAK_ALLOC Fix incorrect CMAKE define SLANG_ENABLE_IR_BREAK_ALLOC, Jun 11, 2025
@ArielG-NV ArielG-NV added the pr: non-breaking PRs without breaking changes label Jun 11, 2025
Copy link
Copy Markdown
Contributor

@gtong-nv gtong-nv left a comment

Choose a reason for hiding this comment

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

LGTM.
Interestingly, I've been using the vs2022-dev preset and was able to get the debug assisting features (Like the "_debugUID") with this define set.

@ArielG-NV ArielG-NV enabled auto-merge (squash) June 11, 2025 20:57
Copy link
Copy Markdown
Collaborator

@jkwak-work jkwak-work left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@ArielG-NV ArielG-NV merged commit c83a6d5 into shader-slang:master Jun 12, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: non-breaking PRs without breaking changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CMake defined SLANG_ENABLE_IR_BREAK_ALLOC, instead of SLANG_ENABLE_IR_BREAK_ALLOC

3 participants