On Tue, Feb 24 2026 at 14:23:57 -05:00:00, Brent W. Baccala
<[email protected]> wrote:
> On Tue, Feb 24, 2026 at 3:16 AM Samuel Thibault
> <[email protected]> wrote:
>
> That's what it thinks, too:
>
>> Our CX8 review was wrong on two points. Damien replied saying the
>> AI was "hallucinating" — and he's right:
>>
>> 1. spl_t is int (4 bytes) on upstream. Our local branch had
>> patched it to unsigned long, and task 72 reviewed our local code,
>> not upstream. The spl.S
>> file doesn't need CX8 conversions.
>> 2. CX16 is impossible in one instruction. x86 addressing modes
>> only support scale factors of 1, 2, 4, 8. The CX(CX()) approach
>> giving 4*4=16 is the
>> correct solution.
>
Ah, yes, classic "You're absolutely right!" response :-)
