Remove impResolveToken from implILConsumesAddr

[EgorBo]

Apply @jakobbotsch's suggestion

[dotnet-policy-service]

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

[EgorBo]

PTAL @jakobbotsch @dotnet/jit-contrib I converted it to a clean up as we discussed, zero diffs.

[Copilot]

Pull Request Overview

This PR updates how the importer checks if an IL instruction consumes an address, removing the old token resolution in favor of a simpler opcode-based check, and adjusts related call sites and helper node creation.

• Changed impILConsumesAddr signature to accept a var_types and codeEndp, and simplified its implementation to handle only CEE_LDFLD.
• Updated call in fgFindJumpTargets to pass the new parameters and remove the redundant struct check.
• Replaced manual clobber-and-rewrite of op1 in impImportBlockCode with a single gtNewLclAddrNode call.
• Updated the function declaration in compiler.h.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
src/coreclr/jit/importer.cpp	Updated impILConsumesAddr signature and body; replaced inline address node rewrite with gtNewLclAddrNode.
src/coreclr/jit/fgbasic.cpp	Adjusted call to impILConsumesAddr to use new parameters and removed the now-unneeded notStruct local.
src/coreclr/jit/compiler.h	Modified impILConsumesAddr declaration to match the new signature.

Comments suppressed due to low confidence (1)

src/coreclr/jit/importer.cpp:53

• [nitpick] Consider renaming the parameter typ to a more descriptive name like operandType or valueType to clarify what type it's representing.

bool Compiler::impILConsumesAddr(var_types typ, const BYTE* codeAddr, const BYTE* codeEndp)