Все, я победил этот странный баг с искаженными цветами
Обновил архив с исходниками и скомпилированными файлами ядра.
Проблема никак не связана с моими правками и проявилась только из-за более новой версии компилятора.
Вот функция, которая читает 16-битное слово из регистра VDP, в нашем случае из регистра данных с указателем в CRAM
/* vdp_ctrl.c */
static unsigned int vdp_68k_data_r_m5(void)
{
uint16 data = 0;
/* Clear pending flag */
pending = 0;
/* Check destination code (CD0-CD3) & CD4 */
switch (code & 0x1F)
{
case 0x00:
{
/* read two bytes from VRAM */
/* .... */
break;
}
case 0x04:
{
/* VSRAM index */
int index = addr & 0x7E;
/* .... */
break;
}
case 0x08:
{
/* Read 9-bit word from CRAM */
data = *(uint16 *)&cram[addr & 0x7E];
/* Unpack 9-bit CRAM data (BBBGGGRRR) to 16-bit bus data (BBB0GGG0RRR0) */
data = ((data & 0x1C0) << 3) | ((data & 0x038) << 2) | ((data & 0x007) << 1);
/* Unused bits are set using data from next available FIFO entry */
data |= (fifo[fifo_idx] & ~0xEEE);
/* .... */
break;
}
case 0x0c: /* undocumented 8-bit VRAM read */
{
/* .... */
break;
}
default:
{
/* Invalid code value (normally locks VDP, hard reset required) */
break;
}
}
/* Increment address register */
addr += reg[15];
/* Return data */
return data;
}
А вот функция чтения 32-битного значения из памяти при эмуляции инструкций CPU 68000, например для инструкций типа "move.l (a0),d1".
Переменная "temp->read16" для данного случая содержит указатель на функцию, приведенную выше.
/* m68kcpu.h */
INLINE uint m68ki_read_32(uint address)
{
cpu_memory_map *temp;
uint val;
m68ki_set_fc(FLAG_S | m68ki_get_address_space()) /* auto-disable (see m68kcpu.h) */
m68ki_check_address_error(address, MODE_READ, FLAG_S | m68ki_get_address_space()) /* auto-disable (see m68kcpu.h) */
temp = &m68ki_cpu.memory_map[((address)>>16)&0xff];
if (temp->read16) val = ((*temp->read16)(ADDRESS_68K(address)) << 16) | ((*temp->read16)(ADDRESS_68K(address + 2)));
else val = m68k_read_immediate_32(address);
#ifdef HOOK_CPU
if (UNLIKELY(cpu_hook))
cpu_hook(HOOK_M68K_R, 4, address, val);
#endif
return val;
}
Найдете ошибку?
Порядок вызовов read16 в строке "val = ((*temp->read16)(ADDRESS_68K(address)) << 16) | ((*temp->read16)(ADDRESS_68K(address + 2)));" не определен. При этом результат зависит от этого порядка, так как внутри read16 -> vdp_68k_data_r_m5 происходит увеличение внутреннего адреса VDP.
В данном случае, компилятор почему-то решил, что выгоднее сначала посчитать правую часть выражения, а затем - левую. В результате в 32-битном значении старшее и младшее слова оказались перепутанными.
Этот баг относится к классу "Undefined Behavior"
