Repository navigation
Translate native F2I instructions with immediate operands - #827
Closed
monsterlag wants to merge 3 commits into
Closed
monsterlag wants to merge 3 commits into
monsterlag wants to merge 3 commits into
Conversation
Normally merge b2765ba, preserving all upstream allocator, depth, mesh, cache and workflow changes. Retain this topic's exact authored assertions. Assisted-by: OpenAI Codex Co-Authored-By: OpenAI Codex <noreply@openai.com>
Owner
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Translate native F2I
0x905immediate operands by materializing the complete 32-bit literal with the existing MOV32I helper, then using the checked register-conversion path. Predicate and literal bits 62/63 are preserved; unsupported widths and modes still fail.Updated on main
b2765ba6through a normal merge, preserving its allocator, depth, mesh and cache changes.Validation at
bcea5bd7: the native F2I fixture, Windows/Linux source checks, SPIR-V checks, Windows renderer/builder and Linux AppImage all pass. Separately, the exact 27 authored cases compiled against the combined private 2.0.6 integration and all emitted modules passed Vulkan1.3 validation; the released lowerer refused905 while existing forms passed.Coverage is lowering, Maxwell decoding/conversion visitor, IR and SPIR-V emission, not full CFG construction or GPU numerical validation. No claim about an individual visible symptom or all remaining shader failures. No game-derived programs or caches are included.
Encoding reference: pinned Mesa NAK encoder.