Skip to content

Add /profile link flag to msquic.dll#5352

Merged
anrossi merged 1 commit intomainfrom
anrossi/profile-link-flag
Sep 12, 2025
Merged

Add /profile link flag to msquic.dll#5352
anrossi merged 1 commit intomainfrom
anrossi/profile-link-flag

Conversation

@anrossi
Copy link
Collaborator

@anrossi anrossi commented Aug 13, 2025

Description

Add /profile link flag to Windows user mode builds. This flag puts extra debug information in the PDB to enable performance analysis of the binary and is required for shipping in some scenarios.

Testing

Existing tests should cover this.

Documentation

N/A

@codecov
Copy link

codecov bot commented Aug 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.37%. Comparing base (ca5c05f) to head (be15c7a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5352      +/-   ##
==========================================
- Coverage   85.90%   85.37%   -0.54%     
==========================================
  Files          59       59              
  Lines       18600    18600              
==========================================
- Hits        15979    15880      -99     
- Misses       2621     2720      +99     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@anrossi anrossi changed the title [EXPERIMENT] Add /profile link flag to msquic.dll Add /profile link flag to msquic.dll Sep 10, 2025
@anrossi anrossi marked this pull request as ready for review September 10, 2025 05:03
@anrossi anrossi requested a review from a team as a code owner September 10, 2025 05:03
@anrossi anrossi added Area: Build Partner: .NET By or For the .NET team labels Sep 10, 2025
@anrossi anrossi moved this to In Progress in Transport Tracker Sep 10, 2025
@anrossi anrossi force-pushed the anrossi/profile-link-flag branch from 95b1417 to be15c7a Compare September 10, 2025 05:06
Copy link
Contributor

@mtfriesen mtfriesen left a comment

Choose a reason for hiding this comment

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

Please add a description to the PR explaining what this does and why.

@anrossi anrossi merged commit 1a1ef81 into main Sep 12, 2025
497 of 503 checks passed
@anrossi anrossi deleted the anrossi/profile-link-flag branch September 12, 2025 17:00
@github-project-automation github-project-automation bot moved this from In Progress to Done in Transport Tracker Sep 12, 2025
anrossi added a commit that referenced this pull request Sep 12, 2025
Copilot AI pushed a commit that referenced this pull request Oct 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Build Partner: .NET By or For the .NET team

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants