JIT: Move remaining xarch floating->integral cast implementation to lowering

[saucecontrol]

This changes the lowering of floating->integral casts to always replace the cast node with HWIntrinsics rather than doing fixups ahead of the cast and leaving the node in place as a self-cast or letting it be handled in codegen. Since the self-cast was not always eliminated in codegen, this results in some size and throughput improvements.

Because the cast is always replaced now, genFloatToIntCast is no longer necessary on xarch.

This is best viewed with whitespace ignored. Most of the changes are simply an extra level of indentation for the pre-AVX10.2 code in LowerCast.

Diffs

[dotnet-policy-service]

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

[saucecontrol]

cc @tannergooding

This is more peeled from #116805, with feedback addressed

[tannergooding]

CC. @dotnet/jit-contrib, @EgorBo for secondary sign-off

This has a couple correctness fixes in addition to the general codegen improvements, so if we're not comfortable taking the whole thing for .NET 10, we likely still need to peel off the fixes that were called out

[tannergooding]

CC. @dotnet/jit-contrib for secondary review

[JulieLeeMSFT]

@jakobbotsch, PTAL.

[Copilot]

Pull Request Overview

This PR refactors the JIT's handling of floating-point to integral casts on xarch by moving all conversion logic to the lowering phase. Previously, some casts were handled in codegen via genFloatToIntCast, but now all floating->integral casts are replaced with HWIntrinsic nodes during lowering. This enables better optimization and eliminates the need for genFloatToIntCast on xarch.

Key changes:

• Floating->integral casts are now always replaced with HWIntrinsic nodes in LowerCast
• New AVX10v2 saturating conversion intrinsics are introduced for direct hardware support
• Fallback paths for pre-AVX10v2 hardware use complex SIMD manipulation for saturation semantics
• The genFloatToIntCast function is removed from xarch codegen

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
src/coreclr/jit/lowerxarch.cpp	Implements complete lowering of floating->integral casts with AVX10v2, AVX512, and fallback paths; replaces cast node with HWIntrinsics
src/coreclr/jit/lower.cpp	Returns early from LowerNode after cast lowering since the node is now removed
src/coreclr/jit/instr.cpp	Removes floating->integral conversion instruction selection since these are now lowered to HWIntrinsics
src/coreclr/jit/hwintrinsiclistxarch.h	Adds new AVX10v2 scalar saturating conversion intrinsics and fixes naming consistency
src/coreclr/jit/hwintrinsiccodegenxarch.cpp	Adds codegen support for new AVX10v2 saturating conversion intrinsics
src/coreclr/jit/hwintrinsic.cpp	Enables AVX10v2_X64 ISA range for 64-bit intrinsics
src/coreclr/jit/gentree.cpp	Fixes intrinsic naming from "Truncation" to "Truncated" for consistency
src/coreclr/jit/codegenxarch.cpp	Removes obsolete genFloatToIntCast function that is no longer needed
src/coreclr/jit/codegenlinear.cpp	Adds xarch guard to make floating->integral casts unreachable on xarch
src/coreclr/jit/CMakeLists.txt	Enables SIMD features for i386 Unix builds to support the new lowering approach

[Copilot]

The cast node is being removed from the block range and replaced with castResult, but there's no check to ensure castResult is not null before attempting the replacement. If any of the code paths in the if/else blocks above fail to set castResult, this will result in undefined behavior. Consider adding an assertion assert(castResult != nullptr) before line 1236 to catch potential logic errors.

Suggested change

	LIR::Use castUse;
	LIR::Use castUse;
	assert(castResult != nullptr);

[saucecontrol]

I don't see this assert as particularly useful, but I'm happy to add it if y'all disagree.

[saucecontrol]

Merged up to resolve conflicts with #121251