libbgi: funcptr-диспетч clip/noclip спрайтовых ядер + снятие src[0]-фикса
Диспетчеризация clip/noclip через указатели _gfx_blit_fn/_gfx_heal_fn (common/_gfx_sprite_fns.c, дефолт clip): gfx_sprite_clip() — теперь модуль, переключает указатели один раз; sprite_update/putsprite/ movesprite зовут через указатель — ветка if(clip) из горячего цикла убрана (съедала половину выигрыша noclip). Программа без вызова gfx_sprite_clip() noclip-ядра не линкует. Замер dev-MAME (16 шаров, uncapped): clip 50 → noclip 60-61 fps (+20-22%). Регресс tests/sprites (A/B PASS), size-check OK (balls −237 Б, sprites −3177 Б — отвязались лишние ядра). Фикс CPU-байта write-триггера (preread + EX AF,AF') снят: точная dev-MAME эмулирует ПЛМ, подавляющую CPU-байт при активном burst'е — подтверждено по байтам VRAM (tests/blitw col0 = GREEN через read_vram MCP-моста). Для heal фикс был избыточен всегда (банк 0x50 перезаписывает dst[0]). Строки фикса оставлены закомментированными в трёх leaf'ах на случай отличий реального железа; шапки и §9а/§9б дизайна обновлены. НА ЖЕЛЕЗЕ ПЕРЕПРОВЕРИТЬ. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
+2
-2
@@ -133,8 +133,8 @@ void _gfx_heal_sprite_noclip(int x, int y, uint8_t w, uint8_t h);
|
||||
* gfx_sprite_clip() перенаправляет их на clip- или noclip-ядро ОДИН раз,
|
||||
* а sprite_update/putsprite/movesprite зовут через указатель — один
|
||||
* call-site, без if в цикле → тесный codegen (полный выигрыш noclip,
|
||||
* см. docs/sprite-api-design.md). Установка по умолчанию (clip on) — в
|
||||
* initgraph. */
|
||||
* см. docs/sprite-api-design.md). Дефолт (clip-ядра) — статическая
|
||||
* инициализация в common/_gfx_sprite_fns.c. */
|
||||
typedef void (*_gfx_blit_fp)(int, int, const void *, int, int,
|
||||
uint8_t, uint8_t);
|
||||
typedef void (*_gfx_heal_fp)(int, int, uint8_t, uint8_t);
|
||||
|
||||
@@ -24,11 +24,14 @@
|
||||
* не рвёт). ВНИМАНИЕ: h ограничен малыми спрайтами — большие блиты
|
||||
* (putimage 256 строк) вызывающий обязан резать на чанки ≤16 строк.
|
||||
*
|
||||
* Квирк CPU-байта write-триггера (см. _bgi_copy_rows_raw): src[0]
|
||||
* предчитывается и подставляется в A через EX AF,AF' — CPU-цикл
|
||||
* LD (DE),A кладёт корректный src[0] при любом банке.
|
||||
* Квирк CPU-байта write-триггера СНЯТ (2026-07-13): на точной dev-MAME
|
||||
* (сборка разработчиков Sprinter) ПЛМ подавляет CPU-байт LD (DE),A при
|
||||
* активном burst'е — утечки в dst[0] нет (проверено по байтам VRAM:
|
||||
* tests/blitw col0 @200,150 = GREEN). Старый фикс (preread src[0] +
|
||||
* EX AF,AF') оставлен закомментированным ниже — восстановить, если
|
||||
* реальное железо поведёт себя как старая MAME. ПЕРЕПРОВЕРИТЬ НА ЖЕЛЕЗЕ.
|
||||
*
|
||||
* Клоббер: AF/AF'/BC/DE/HL; IX сохраняется.
|
||||
* Клоббер: AF/BC/DE/HL; IX сохраняется.
|
||||
*/
|
||||
|
||||
#include "../_bgi.h"
|
||||
|
||||
@@ -33,20 +33,19 @@
|
||||
* LD (DE),A (burst-запись из памяти акселератора в dst), LD B,B, EI.
|
||||
* Между армированием и триггерами — только регистровые M1-опкоды.
|
||||
*
|
||||
* КВИРК ТРИГГЕРА ЗАПИСИ (найден tests/sprites — клипнутые спрайты):
|
||||
* LD (DE),A — реальная CPU-инструкция: её собственный цикл записи
|
||||
* кладёт байт A в dst[0], и лишь затем burst FSM пишет src[0..w-1]
|
||||
* поверх. После burst-чтения в A остаётся ПОСЛЕДНИЙ байт строки —
|
||||
* при банке 0x50 утечка невидима (src[0] тут же перезапишет), но при
|
||||
* 0x58/0x5C, если src[0] == 0xFF, перезапись скипается и в dst[0]
|
||||
* остаётся вертикальная полоса цвета последних байтов строк.
|
||||
* Нейтрализация: src[0] предчитывается ДО армирования и подставляется
|
||||
* в A через EX AF,AF' (регистровый M1-опкод — под armed разрешён)
|
||||
* прямо перед триггером — CPU-байт становится src[0], что корректно
|
||||
* при ЛЮБОМ банке и любой семантике первого цикла записи (в MAME
|
||||
* 0.283 подтверждено tests/blitw+sprites; на железе перепроверить).
|
||||
* КВИРК ТРИГГЕРА ЗАПИСИ — ФИКС СНЯТ 2026-07-13. История: в стоковой
|
||||
* MAME 0.283 LD (DE),A вёл себя как реальная CPU-инструкция — её
|
||||
* собственный цикл записи клал байт A (последний байт строки после
|
||||
* burst-чтения) в dst[0], и при банке 0x58/0x5C со src[0] == 0xFF
|
||||
* оставалась вертикальная полоса (найдено tests/sprites).
|
||||
* Нейтрализация была: preread src[0] + EX AF,AF' перед триггером.
|
||||
* Точная dev-MAME (сборка разработчиков Sprinter) эмулирует ПЛМ,
|
||||
* подавляющую CPU-байт при активном burst'е — утечки нет и без фикса
|
||||
* (проверено tests/blitw: col0 @200,150 = GREEN, по байтам VRAM).
|
||||
* Закомментированные строки ниже — след фикса; восстановить, если
|
||||
* реальное железо поведёт себя как MAME 0.283. НА ЖЕЛЕЗЕ ПЕРЕПРОВЕРИТЬ.
|
||||
*
|
||||
* Клоббер: AF/AF'/BC/DE/HL; IX сохраняется (callee-saved).
|
||||
* Клоббер: AF/BC/DE/HL; IX сохраняется (callee-saved).
|
||||
*/
|
||||
|
||||
#include "../_bgi.h"
|
||||
|
||||
@@ -15,12 +15,15 @@
|
||||
* вызывающий (gfx_heal форсит 0x50 — иначе heal бессмыслен).
|
||||
* Полосы > 256 байт режет вызывающий.
|
||||
*
|
||||
* Квирк CPU-байта write-триггера (см. _bgi_copy_rows_raw): для heal
|
||||
* предчитанный src[0] — ровно тот байт, что burst запишет в dst[0],
|
||||
* так что фикс EX AF,AF' здесь одновременно и нейтрализация утечки,
|
||||
* и просто корректное значение.
|
||||
* Квирк CPU-байта write-триггера: фикс (preread + EX AF,AF') СНЯТ
|
||||
* (2026-07-13). Для heal он был избыточен ВСЕГДА: банк 0x50 без
|
||||
* прозрачности — даже если CPU-байт утёк бы в dst[0], burst тут же
|
||||
* перезаписывает его тем же значением из ОЗУ-копии. Плюс точная
|
||||
* dev-MAME показала, что ПЛМ вообще подавляет CPU-байт (см.
|
||||
* _bgi_blit_rows_raw). Закомментированные строки ниже — след старого
|
||||
* фикса на случай отличий реального железа.
|
||||
*
|
||||
* Клоббер: AF/AF'/BC/DE/HL; IX сохраняется.
|
||||
* Клоббер: AF/BC/DE/HL; IX сохраняется.
|
||||
*/
|
||||
|
||||
#include "../_bgi.h"
|
||||
|
||||
@@ -35,7 +35,6 @@
|
||||
* умножения: обычно sy == 0).
|
||||
*/
|
||||
#include "../_bgi.h"
|
||||
#include <stdio.h>
|
||||
|
||||
void _gfx_blit_sprite(int x, int y, const void *img,
|
||||
int sx, int sy, uint8_t w, uint8_t h)
|
||||
|
||||
@@ -0,0 +1,16 @@
|
||||
/*
|
||||
* _gfx_sprite_fns — указатели диспетчеризации clip/noclip спрайтовых ядер
|
||||
* (data-модуль). Дефолт — clip-ядра (безопасно); gfx_sprite_clip()
|
||||
* перенаправляет на noclip и обратно. Горячие call-site'ы
|
||||
* (sprite_update/putsprite/movesprite) зовут через указатель — один
|
||||
* call-site без if в цикле → тесный codegen (ветка if(clip) в
|
||||
* sprite_update съедала половину выигрыша noclip: 52 вместо 57 fps).
|
||||
*
|
||||
* Программа, не зовущая gfx_sprite_clip(), тянет только clip-ядра
|
||||
* (noclip-ядра referenced лишь из модуля gfx_sprite_clip.c — DCE).
|
||||
* Ненулевой дефолт → в _INITIALIZED (как _gfx_bank).
|
||||
*/
|
||||
#include "../_bgi.h"
|
||||
|
||||
_gfx_blit_fp _gfx_blit_fn = _gfx_blit_sprite;
|
||||
_gfx_heal_fp _gfx_heal_fn = _gfx_heal_sprite;
|
||||
@@ -0,0 +1,25 @@
|
||||
/*
|
||||
* gfx_sprite_clip — включить/выключить клип спрайтов по экрану.
|
||||
* on=1 (дефолт) — clip-ядра; on=0 — noclip-ядра (приложение ГАРАНТИРУЕТ,
|
||||
* что спрайты целиком на экране; ~+19% на анимации — клип-кода в noclip
|
||||
* нет физически, см. _gfx_blit_sprite_noclip.c).
|
||||
*
|
||||
* Переключение — перенаправлением указателей _gfx_blit_fn/_gfx_heal_fn
|
||||
* (ОДИН раз здесь, не ветка в горячем цикле). Флаг _gfx_sprite_clip
|
||||
* остаётся для gfx_get_sprite_clip(). Настоящая функция (не inline):
|
||||
* только этот модуль ссылается на noclip-ядра — программа без вызова
|
||||
* gfx_sprite_clip() их не линкует.
|
||||
*/
|
||||
#include "../_bgi.h"
|
||||
|
||||
void gfx_sprite_clip(uint8_t on)
|
||||
{
|
||||
_gfx_sprite_clip = on;
|
||||
if (on) {
|
||||
_gfx_blit_fn = _gfx_blit_sprite;
|
||||
_gfx_heal_fn = _gfx_heal_sprite;
|
||||
} else {
|
||||
_gfx_blit_fn = _gfx_blit_sprite_noclip;
|
||||
_gfx_heal_fn = _gfx_heal_sprite_noclip;
|
||||
}
|
||||
}
|
||||
@@ -22,8 +22,7 @@ void movesprite(int oldx, int oldy, int x, int y, const void *img)
|
||||
|
||||
gfx_set_bank(GFX_BANK_NORMAL);
|
||||
_bgi_begin();
|
||||
if (_gfx_sprite_clip) _gfx_heal_sprite(oldx, oldy, p[0], p[2]);
|
||||
else _gfx_heal_sprite_noclip(oldx, oldy, p[0], p[2]);
|
||||
(*_gfx_heal_fn)(oldx, oldy, p[0], p[2]);
|
||||
_bgi_end();
|
||||
gfx_set_bank(saved);
|
||||
|
||||
|
||||
@@ -18,8 +18,7 @@ void putsprite(int x, int y, const void *img)
|
||||
|
||||
gfx_set_bank(GFX_BANK_SPRITE);
|
||||
_bgi_begin();
|
||||
if (_gfx_sprite_clip) _gfx_blit_sprite(x, y, img, 0, 0, p[0], p[2]);
|
||||
else _gfx_blit_sprite_noclip(x, y, img, 0, 0, p[0], p[2]);
|
||||
(*_gfx_blit_fn)(x, y, img, 0, 0, p[0], p[2]);
|
||||
_bgi_end();
|
||||
gfx_set_bank(saved);
|
||||
}
|
||||
|
||||
@@ -31,7 +31,6 @@ void sprite_update(sprite_t *arr, uint8_t count)
|
||||
uint8_t p = gfx_get_draw_page(); /* 0 / 1 */
|
||||
uint8_t dbit = p ? _SPR_DIRTY1 : _SPR_DIRTY0;
|
||||
uint8_t saved = gfx_get_bank();
|
||||
uint8_t clip = _gfx_sprite_clip; /* раз на кадр: clip- или noclip-ядро */
|
||||
uint8_t i;
|
||||
sprite_t *s;
|
||||
|
||||
@@ -40,8 +39,7 @@ void sprite_update(sprite_t *arr, uint8_t count)
|
||||
_bgi_begin();
|
||||
for (i = 0, s = arr; i < count; i++, s++) {
|
||||
if (s->drawn[p].on && (s->flags & (dbit | SPR_ALWAYS))) {
|
||||
if (clip) _gfx_heal_sprite(s->drawn[p].x, s->drawn[p].y, s->w, s->h);
|
||||
else _gfx_heal_sprite_noclip(s->drawn[p].x, s->drawn[p].y, s->w, s->h);
|
||||
(*_gfx_heal_fn)(s->drawn[p].x, s->drawn[p].y, s->w, s->h);
|
||||
s->drawn[p].on = 0;
|
||||
}
|
||||
}
|
||||
@@ -53,8 +51,7 @@ void sprite_update(sprite_t *arr, uint8_t count)
|
||||
for (i = 0, s = arr; i < count; i++, s++) {
|
||||
if (s->flags & (dbit | SPR_ALWAYS)) {
|
||||
if (s->flags & SPR_VISIBLE) {
|
||||
if (clip) _gfx_blit_sprite(s->x, s->y, s->img, s->sx, s->sy, s->w, s->h);
|
||||
else _gfx_blit_sprite_noclip(s->x, s->y, s->img, s->sx, s->sy, s->w, s->h);
|
||||
(*_gfx_blit_fn)(s->x, s->y, s->img, s->sx, s->sy, s->w, s->h);
|
||||
s->drawn[p].x = s->x;
|
||||
s->drawn[p].y = s->y;
|
||||
s->drawn[p].on = 1;
|
||||
|
||||
@@ -70,10 +70,11 @@ inline uint8_t gfx_get_bank(void) { return _gfx_bank; }
|
||||
* иначе запись за край портит соседнюю память, см. tests/spriteclip).
|
||||
* Выигрыш ~+19% на анимации: движок зовёт noclip-ядра без клип-кода
|
||||
* (не просто скип проверки — вся клип-математика физически отсутствует,
|
||||
* SDCC тесней раскладывает регистры). Флаг читается раз на спрайт.
|
||||
* НЕ влияет на putimage/gfx_blit (у них клип всегда). */
|
||||
* SDCC тесней раскладывает регистры). Переключение — перенаправлением
|
||||
* внутренних указателей на ядра (один раз здесь, не ветка в горячем
|
||||
* цикле движка). НЕ влияет на putimage/gfx_blit (у них клип всегда). */
|
||||
extern uint8_t _gfx_sprite_clip;
|
||||
inline void gfx_sprite_clip(uint8_t on) { _gfx_sprite_clip = on; }
|
||||
void gfx_sprite_clip(uint8_t on);
|
||||
inline uint8_t gfx_get_sprite_clip(void) { return _gfx_sprite_clip; }
|
||||
|
||||
/* ---- Блиттинг (ядро спрайтов; Фаза B sprite-api-design) ----------- *
|
||||
|
||||
Reference in New Issue
Block a user