Skip to content

Remove some EDG workarounds - #6474

Open
S. B. Tam (cpplearner) wants to merge 4 commits into
microsoft:mainfrom
cpplearner:edg-case
Open

S. B. Tam (cpplearner) wants to merge 4 commits into
microsoft:mainfrom
cpplearner:edg-case

Conversation

@cpplearner

Copy link
Copy Markdown
Contributor

EDG's change log indicates that the relevant bugs have been fixed.

No functional change.

Copilot AI balanced review requested due to automatic review settings October 2, 2026 02:40
@cpplearner
S. B. Tam (cpplearner) requested a review from a team as a code owner October 2, 2026 02:40
@azure-pipelines

This comment was marked as resolved.

This comment was marked as resolved.

Copilot AI balanced review requested due to automatic review settings October 2, 2026 03:13

This comment was marked as resolved.

@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) added the enhancement Something can be improved label Oct 2, 2026
… requires-expressions" was fixed.

It was resolved as fixed by Xinyu Sui on 2026-08-24 with "This was actually fixed, likely by VS-PR-761263 'EDG 6.9'" (merged 2026-08-05).
…igStack::sigStack.size() == 1" was fixed.

JonCaves fixed this with MSVC-PR-732611 on 2026-04-27, then added a regression test with MSVC-PR-733951 on 2026-04-30.
Copilot AI balanced review requested due to automatic review settings October 3, 2026 12:30

This comment was marked as resolved.

@StephanTLavavej

Copy link
Copy Markdown
Member

One more workaround can be removed! It evaded your careful search because it never got an EDGcpfe bug number. And coincidentally, MSVC was also fixed.

This branch has not been deployed

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

Labels

enhancement Something can be improved

Projects

Status: Ready To Merge

Development

Successfully merging this pull request may close these issues.

3 participants