Skip to content

[Minor] Refactor MSBuild project structure#2209

Merged
Belonit merged 2 commits into
developfrom
refactor-msbuild-project-structure
May 14, 2026
Merged

[Minor] Refactor MSBuild project structure#2209
Belonit merged 2 commits into
developfrom
refactor-msbuild-project-structure

Conversation

@Belonit
Copy link
Copy Markdown
Member

@Belonit Belonit commented May 12, 2026

Refactors the MSBuild project layout and prepares the build configuration for safer optimized builds.

Changes:

  • Split Phobos build settings into Phobos.props.
  • Import project-wide props from Phobos.vcxproj.
  • Cleaned up duplicated and redundant MSBuild settings.
  • Prepared linker retention settings for WPO/LTCG compatibility.

Note:

The macro changes are a workaround needed to prevent the linker from removing records from custom sections such as .patch, .syexe00, and .syhks00.

Without these anchors, the linker may treat those symbols as unused and discard them when WPO/LTCG is enabled.

@Belonit Belonit added Minor Minor feature and/or fix, not a lot of changes or they are not significant No Documentation Needed No documentation needed whatsoever and removed Minor Minor feature and/or fix, not a lot of changes or they are not significant labels May 12, 2026
@github-actions
Copy link
Copy Markdown

Nightly build for this pull request:

This comment is automatic and is meant to allow guests to get latest nightly builds for this pull request without registering. It is updated on every successful build.

@Belonit Belonit requested a review from Metadorius May 13, 2026 00:05
@Belonit Belonit merged commit 600ad42 into develop May 14, 2026
6 of 30 checks passed
@Belonit Belonit deleted the refactor-msbuild-project-structure branch May 14, 2026 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Minor Minor feature and/or fix, not a lot of changes or they are not significant No Documentation Needed No documentation needed whatsoever

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant