snark13/Sprinter-SDCC
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

libbgi: NOT_PUT тоже через акселератор + изыскания по виду Тени

Browse Source 
NOT.  У акселератора нет режима «инвертировать буфер»: буфер меняется
только на ЧТЕНИИ и только опкодами AND/OR/XOR (HL) (драйвер MAME,
update_accel_buffer), а `CPL` автомат вообще не распознаёт — инвертируется
регистр CPU, не буфер.  Зато ~src = src XOR #FF, поэтому NOT собирается из
уже имеющегося: буфер := src, XOR-burst по константному блоку единиц
(common/_bgi_ones256.c), запись.  Ядра _bgi_blit_{rows,cols}_not_raw.c;
между burst'ами меняется HL, поэтому каждая смена — под СТОПом (иначе fetch
операнда перезапустит burst).  В колоночном варианте второй OUT Port_Y не
нужен: op-burst идёт по блоку единиц горизонтально, а Port_Y шагает только
вертикальный.

Наружу — тем же op-параметром (GFX_OP_NOT), диспетчер один раз на вызов, в
цикл по полосам/колонкам не заходит.  putimage лишился попиксельного пути
ЦЕЛИКОМ: все пять операций BGI идут через акселератор и клиппируются
одинаково.  tests/accop дополнен T9/T10 (NOT строками и колонками) —
10/10 PASS в MAME; tests/bgi_img P1/P2 по-прежнему PASS.

tests/convbench (новый) — замер побайтной конвертации атласа
«прозрачный #FF -> 0x00» (источник для XOR-блита): 145.3 такта/байт, то
есть 2.5× от 59 номинальных T-states цикла.  0.11 с на страницу 16 КБ,
3.2 с на все 28 страниц, 1.3 с по фактическому объёму данных (186 КБ).

applications/PoP/docs/shadow_render.md — вид Тени (два блиттера OR+XOR)
ОТЛОЖЕН по решению пользователя: пока рисуем обычной копией атласами Кида.
В документе собрано, почему в лоб не выходит (XOR несовместим с
прозрачностью #FF; операция читает ОЗУ-копию, поэтому два прохода
оригинала вырождаются в один XOR — нужен однопроходный композит
s | bg ^ s(сдвиг)), замер стоимости источника и четыре варианта.  Ключ к
выбору — список кадров, которыми тень реально пользуется; снимать по факту,
когда пойдут уровни 5/6/12.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Alexander Petrov
2026-08-11 15:41:06 +03:00
parent a823e7ee9c
commit 860468f3c5
15 changed files with 616 additions and 35 deletions
Download Patch File Download Diff File Expand all files Collapse all files
libbgi/common/_bgi_ones256.c
+53
View File
@@ -0,0 +1,53 @@
/*
* _bgi_ones256 — константный блок из 256 байт #FF: второй операнд для
* блит-операции NOT (GFX_OP_NOT).
*
* ЗАЧЕМ. У акселератора нет режима «инвертировать буфер»: буфер меняется
* только на ЧТЕНИИ и только опкодами AND/OR/XOR (HL) (см. драйвер MAME,
* update_accel_buffer). `CPL` автомат не распознаёт вовсе — он
* инвертирует регистр CPU, а не буфер. Зато ~src = src XOR #FF, поэтому
* NOT собирается из уже имеющегося: буфер := src, потом XOR-burst по
* этому блоку, потом запись.
*
* Отдельным модулем — по правилу «общие статики в свой модуль»: блок
* нужен обоим ядрам (rows и cols), а программа без NOT его не линкует.
* const => живёт в _CODE, то есть вне W3 — как и требует accel (сторона
* ОЗУ на время операции не должна быть перекрыта видеобанком).
* Размер ровно 256 — максимальный размер accel-блока.
*/
#include "../_bgi.h"
const uint8_t _bgi_ones256[256] = {
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF,
0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF, 0xFF
};
libbgi/common/_gfx_blit_full_op.c
+15
View File
@@ -67,6 +67,21 @@ void _gfx_blit_full_op(int x, int y, const void *img,
src += (uint16_t)sy * (uint16_t)img_w;
dst = _gfx_addr_base + x;
/* NOT — не операция с приёмником, а инверсия источника, и цепочка у
* неё другая (XOR по блоку единиц вместо чтения приёмника), поэтому
* своё ядро. Ветка одна на вызов, в цикл по полосам не заходит. */
if (op == GFX_OP_NOT) {
while (w > 256) {
_bgi_blit_rows_not_raw(src, (uint8_t *)dst, 0 /* =256 */,
(uint8_t)h, img_w, (uint8_t)y);
src += 256;
dst += 256;
w -= 256;
}
_bgi_blit_rows_not_raw(src, (uint8_t *)dst, (uint8_t)w, (uint8_t)h,
img_w, (uint8_t)y);
return;
}
while (w > 256) {
_bgi_blit_rows_op_raw(src, (uint8_t *)dst, 0 /* =256 */, (uint8_t)h,
img_w, (uint8_t)y, op);
libbgi/common/gfx_blit_cols_part_wx_op.c
+8 -2
View File
@@ -89,7 +89,13 @@ void gfx_blit_cols_part_wx_op(int x, int y, const void *img, uint8_t flip,
_bgi_begin();
dst = (uint8_t *)(_gfx_addr_base + (uint16_t)x);
_bgi_blit_cols_op_raw(src, dst, (uint8_t)w, (uint8_t)h, sstride,
(uint8_t)y, op);
/* NOT — инверсия ИСТОЧНИКА, приёмник в ней не участвует: своя цепочка
* (XOR по блоку единиц) и своё ядро. Ветка одна на вызов. */
if (op == GFX_OP_NOT)
_bgi_blit_cols_not_raw(src, dst, (uint8_t)w, (uint8_t)h, sstride,
(uint8_t)y);
else
_bgi_blit_cols_op_raw(src, dst, (uint8_t)w, (uint8_t)h, sstride,
(uint8_t)y, op);
_bgi_end();
}
libbgi/common/putimage.c
+15 -25
View File
@@ -8,17 +8,16 @@
* с учётом текущего банка (gfx_set_bank: 0x58 даёт аппаратную
* прозрачность 0xFF, 0x54/0x5C — временный вывод).
*
* XOR/OR/AND_PUT — тоже через акселератор (gfx_blit_part_op, 2026-08-11;
* закрыт пункт 2d-1 docs/TODO.md): акселератор умеет блочные AND/OR/XOR
* между своим буфером и памятью приёмника, попиксельного цикла CPU больше
* нет. Клиппинг у них теперь ТОТ ЖЕ, что у COPY_PUT (раньше на этом пути
* клипа не было вовсе). Оговорки — в шапке common/_gfx_blit_full_op.c:
* операция читает ОЗУ-копию экрана (при банках 0x54/0x5C это чистый фон),
* а прозрачность #FF совместима с AND/OR, но не с XOR.
* XOR/OR/AND/NOT_PUT — тоже через акселератор (gfx_blit_part_op,
* 2026-08-11; закрыт пункт 2d-1 docs/TODO.md): у железа есть блочные
* AND/OR/XOR между буфером акселератора и памятью приёмника, а NOT
* выражается через XOR по блоку единиц. Попиксельного пути в putimage
* больше НЕТ ВООБЩЕ, и клиппинг у всех пяти операций теперь одинаковый
* (раньше на непрямом пути его не было вовсе).
*
* NOT_PUT — по-прежнему по-пиксельно (пишем ~src): это не операция с
* приёмником, а инверсия источника, у акселератора такого режима нет.
* Клиппинга на этом пути НЕТ (per-pixel bounds-check только в safe).
* Оговорки — в шапке common/_gfx_blit_full_op.c: операция читает
* ОЗУ-копию экрана (при банках 0x54/0x5C это чистый фон, а не то, что
* нарисовано поверх), а прозрачность #FF совместима с AND/OR, но не с XOR.
*
* Буфер bitmap обязан лежать вне W3 (< 0xC000) — на время операции W3
* замаплен на видеобанк.
@@ -28,7 +27,7 @@
void putimage(int left, int top, const void *bitmap, int op)
{
const uint8_t *p = (const uint8_t *)bitmap;
int w, h, x, y;
int w, h;
uint8_t gop;
w = p[0] | (p[1] << 8);
@@ -36,23 +35,14 @@ void putimage(int left, int top, const void *bitmap, int op)
if (w <= 0 || h <= 0) return;
switch (op) {
case COPY_PUT:
gfx_blit_part(left, top, bitmap, 0, 0, w, h);
return;
case XOR_PUT: gop = GFX_OP_XOR; break;
case OR_PUT: gop = GFX_OP_OR; break;
case AND_PUT: gop = GFX_OP_AND; break;
default: gop = 0; break; /* NOT_PUT — попиксельно ниже */
}
if (gop) {
gfx_blit_part_op(left, top, bitmap, 0, 0, w, h, gop);
case NOT_PUT: gop = GFX_OP_NOT; break;
case COPY_PUT:
default:
gfx_blit_part(left, top, bitmap, 0, 0, w, h);
return;
}
p += 4;
_bgi_begin();
for (y = 0; y < h; y++)
for (x = 0; x < w; x++)
_bgi_plot_raw(left + x, top + y, (uint8_t)~(*p++));
_bgi_end();
gfx_blit_part_op(left, top, bitmap, 0, 0, w, h, gop);
}