Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enables the interpreter on iOS #44911

Merged
merged 4 commits into from
Nov 19, 2020
Merged

Enables the interpreter on iOS #44911

merged 4 commits into from
Nov 19, 2020

Conversation

steveisok
Copy link
Member

@steveisok steveisok commented Nov 19, 2020

To enable on tests, you can pass MonoForceInterpreter=true as an extra MSBuild property.

AppleAppBuilder will also have a ForceInterpreter property on it in order to flow down to device / simulator.

Steve Pfister added 2 commits November 19, 2020 09:11
To enable on tests, you can pass MonoForceInterpreter=true as an extra MSBuild property.

AppleAppBuilder will also have a ForceInterpreter property on it in order to flow down to device / simulator.  Additionally, removed eventpipe from iOS when
TargetsiOSSimulator == true as it was asserting for debug builds.
…enable the debug configuration to work for the time being
Copy link
Member

@lateralusX lateralusX left a comment

Choose a reason for hiding this comment

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

With diagnostic server assert out of the way, shouldn't we enable eventpipe on all IOS configurations?

@lateralusX
Copy link
Member

Looks like that triggered a build error on older IOS SDK versions. Then we should just enable the simulator for now and I can look/fix that in separate PR and enable the rest of IOS platforms.

@steveisok steveisok merged commit b851613 into dotnet:master Nov 19, 2020
@steveisok steveisok deleted the ios-interp branch November 19, 2020 23:40
@ghost ghost locked as resolved and limited conversation to collaborators Dec 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants