JIT-EE interface clean up (tracking issue)

[EgorBo]

I suggest to use this issue to store ideas of what we can clean up in the JIT-EE interface. So, when someone modifies it they can address these issues along the way to avoid bumping JIT-EE guid for single minor things:

To do:

• recordRelocation has unused arg slotNum
• isCompatibleDelegate is unused
• mergeClasses is unused
• satisfiesClassConstraints is unused
• findNameOfToken is unused
• getInlinedCallFrameVptr is unused
• canAccessFamily is unused
• isRIDClassDomainID is unused
• isValidToken is unused (verification artifact?)
• isValidStringRef is unused (verification artifact?)
• tryResolveToken is unused
• CORINFO_DESKTOP_ABI
• getMethodModule

Initially, I wanted to file this issue with just 2 items but found more.

cc @dotnet/jit-contrib @jkotas

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch
See info in area-owners.md if you want to be subscribed.

Issue Details

---

I suggest to use this issue to store ideas of what we can clean up in the JIT-EE interface. So, when someone modifies it they can address these issues along the way to avoid bumping JIT-EE guid for single minor things:

To do:

• recordRelocation has unused arg slotNum
• isCompatibleDelegate is unused
• mergeClasses is unused
• satisfiesClassConstraints is unused
• findNameOfToken is unused
• getInlinedCallFrameVptr is unused
• canAccessFamily is unused
• isRIDClassDomainID is unused
• isValidToken is unused (verification artificat?)
• isValidStringRef is unused (verification artificat?)

To discuss:

• classMustBeLoadedBeforeCodeIsRun and methodMustBeLoadedBeforeCodeIsRun are likely some NGen artifacts?

Initially, I wanted to file this issue with just 2 items but found more.

cc @dotnet/jit-contrib @jkotas

Author:	EgorBo
Assignees:	-
Labels:	area-CodeGen-coreclr
Milestone:	-

[jkotas]

classMustBeLoadedBeforeCodeIsRun and methodMustBeLoadedBeforeCodeIsRun are likely some NGen artifacts?

These are still used by R2R compiler. It may be possible to get rid of them by refactoring the protocol, but I do not think it is safe to just delete them.

[MichalPetryka]

getMethodModule also seems to be unused.

[BruceForstall]

Fixed with #87588