Skip to content

Conversation

@CyrusNajmabadi
Copy link
Member

@CyrusNajmabadi CyrusNajmabadi commented Aug 26, 2025

The PR this change went in with seems to have a high regression. This means one of two things (at least):

  1. there is something that is very bad about calling into oop here that we don't understand yet.
  2. the logic is just overall busted, regardless of if we are calling into oop.

This PR at least tests if '1' is the issue. If so, we can just disable the oop call and unblock things. If this doesn't address the problem, then something deeper is in effect and we'llhave to revert #79991 entirely. That's going to be a painful change (due to subsequent prs going on), so it would be nice to avoid.

Build: https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=12269021&view=results
PR: https://dev.azure.com/devdiv/DevDiv/_git/VS/pullrequest/665108

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.

2 participants