Skip to content

Conversation

@kouvel
Copy link
Contributor

@kouvel kouvel commented Feb 9, 2019

  • For scenarios that don't mind giving up some startup time (but not too much), it may be a reasonable option when it is desirable to avoid spending two JIT cycles on methods
  • I'd like to use this mode as the default for microbenchmarks in the performance repo and the most convenient way of doing so would be to have a way to specify it in a project file, to keep benchmarks easily runnable with dotnet run. It serves as a soft workaround to https://github.com/dotnet/coreclr/issues/19751 while still allowing perf improvements from tiering to show when R2R is being used, in benchmarks that could help to get more representative numbers.

- For scenarios that don't mind giving up some startup time (but not too much), it may be a reasonable option when it is desirable to avoid spending two JIT cycles on methods
- I'd like to use this mode as the default for microbenchmarks in the performance repo and the most convenient way of doing so would be to have a way to specify it in a project file. It serves as a soft workaround to https://github.com/dotnet/coreclr/issues/19751 while still allowing perf improvements from tiering to show when R2R is being used, in benchmarks that could help to get more representative numbers.
@kouvel
Copy link
Contributor Author

kouvel commented Feb 9, 2019

@noahfalk @livarcocc

@nguerrera
Copy link
Contributor

Do you want this in 3.0 Preview 3? If so, retarget to release/3.0.1xx branch.

@kouvel
Copy link
Contributor Author

kouvel commented Feb 11, 2019

Thanks, it's not necessary to be in 3.0 preview 3

@nguerrera nguerrera added this to the 3.0.1xx milestone Feb 11, 2019
@nguerrera nguerrera merged commit 0ddc87b into dotnet:master Feb 11, 2019
@kouvel kouvel deleted the DisableTier0JitSdk branch February 11, 2019 15:53
dsplaisted pushed a commit to dsplaisted/sdk that referenced this pull request Feb 19, 2020
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