Skip to content

Conversation

@BruceForstall
Copy link
Contributor

Also, update some of the text, and fix an issue with the useLegacyJit config file syntax.

Also, update some of the text, and fix an issue with the useLegacyJit config file syntax.
@BruceForstall
Copy link
Contributor Author

@richlander @cmckinsey PTAL

Choose a reason for hiding this comment

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

"Disabling the use of NGEN may result in significantly slower application startup times"

@cmckinsey
Copy link

Just the one change for the above.

Should we also include a section on "COMPLUS_JitMinOpts" along the methodimpl suggestion for broader opt disabling rather than just the one?

@BruceForstall
Copy link
Contributor Author

@cmckinsey I updated the PR; PTAL

@cmckinsey
Copy link

Thanks for the updates. LGTM.

@cmckinsey
Copy link

Should this be in docs section here in dotnet or dotnet\coreclr?

@BruceForstall
Copy link
Contributor Author

I'm just changing the doc the Rich started here. I'm not entirely sure what this repo is intended for. Also, we can't easily change the location (or even name of the doc or name of the sections) because it is the target of a number of external links now (I'm not sure where or how many at this point).

BruceForstall added a commit that referenced this pull request Jul 30, 2015
@BruceForstall BruceForstall merged commit 6565254 into microsoft:master Jul 30, 2015
Copy link
Member

Choose a reason for hiding this comment

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

This isn't true anymore, "Suppress JIT optimizations on module load" is off by default in 2015.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Apparently so. The referenced MSDN document is wrong, then.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants