Skip to content

JIT: Improve inliner - static readonly fields, pinvokes, Span::Length #62421

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.

Sign up for GitHub

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

Closed
EgorBo wants to merge 11 commits into from
Closed

JIT: Improve inliner - static readonly fields, pinvokes, Span::Length #62421

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
4b9429a
Improvements for the inliner, part 3:
EgorBo Dec 5, 2021
32c78c0
Merge branch 'main' of https://github.com/dotnet/runtime into jit-inl…
EgorBo Dec 14, 2021
69cea24
Add CONST_ARG_SETS_FLD heuristic to avoid write-barriers with const args
EgorBo Dec 14, 2021
f69b81a
formatting
EgorBo Dec 14, 2021
8bb5ddc
Merge branch 'main' into jit-inliner-part-3
EgorBo Dec 23, 2021
8e8f24d
Fix regressions
EgorBo Dec 23, 2021
675edde
recognize constants in isinst(exact-arg)
EgorBo Dec 24, 2021
195bd3f
Tuning
EgorBo Dec 24, 2021
e3514ec
Merge branch 'main' of https://github.com/dotnet/runtime into jit-inl…
EgorBo Dec 24, 2021
f7c3276
Revert some changes
EgorBo Dec 24, 2021
2838529
Clean up
EgorBo Dec 24, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
formatting
  • Loading branch information
@EgorBo
EgorBo committed Dec 14, 2021
commit f69b81a9078b96b245ce09a5e75f773f62426ab3
3 changes: 2 additions & 1 deletion src/coreclr/jit/inlinepolicy.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1516,7 +1516,8 @@ double ExtendedDefaultPolicy::DetermineMultiplier()
// What it means that for this specific st(s)fld we can avoid emitting a heavy write-barrier
// if we inline the candidate
multiplier += 1.5 + m_ConstArgSetsFld;
JITDUMP("\nInline candidate has const arg that sets %d fields. Multiplier increased to %g.", m_ConstArgSetsFld, multiplier);
JITDUMP("\nInline candidate has const arg that sets %d fields. Multiplier increased to %g.", m_ConstArgSetsFld,
multiplier);
}

if ((m_FoldableBox > 0) && m_NonGenericCallsGeneric)
Expand Down