Code-quality review on 25eba79ad found two Important issues:
- launch_sp11_novelty_simhash_lookup + launch_sp11_novelty_simhash_update
were defined with #[allow(dead_code)] since B1 hasn't wired them yet —
feedback_no_hiding violation. Deleted both functions; kept the kernel
handle fields + cubin loads + MappedF32Buffer storage. B1 will inline
the launches at the actual call site (matching A0's deferral pattern
for the novelty-hash registry entry). Field doc-refs at the kernel
declarations updated so no stale name reference remains.
- Seed XOR-fold comment named what ("fold the high + low halves") but
not why ("to preserve entropy; truncation would silently discard
upper 32 bits"). Extended comment so a future reader switching to
truncation gets the warning.
Verified: cargo check clean (18 unrelated warnings, zero errors); 6/6
SP11 GPU oracle tests build under --features cuda (GPU-gated, ignored
on CPU runners); 10/10 sp5_isv_slots + 4/4 state_reset_registry
contract tests pass; allow(dead_code) count is exactly the 12
pre-existing attributes (was 14 before this fix; A2 added 2).
44fb4531a8