Оптимизация логики: окно коллизии как в оригинале + деление таблицей
Замерено брейкпоинтами в MAME (уровень 1 комната 1, Кид стоит у факела). Калибровка, без которой цифры не сходятся: такт totalcycles != номинальный T-такт Z80, wait-state'ы ОЗУ Sprinter дают ~2,4x (get_tile 574 против 1422). 1. Окно перебора коллизии — как у оригинала (left_checked_col..right_checked_col, seg004:0047), было: все 14 колонок каждый кадр. Признак годности слота у нас дешевле оригинального: не массив номеров комнат с очисткой, а границы окна, которые move_coll_to_prev переносит в prev вместе с флагами; бамп считается по пересечению двух окон. check_chomped_flags тоже ограничен окном, иначе протухшие слоты дают фантомный перемол. 2. get_tile_div_mod — таблицами tile_div_tbl/tile_mod_tbl (seg006:702), было /14 и %14. SDCC разворачивал это в __divsint + __modsint, а __modsint внутри зовёт __divsint ещё раз: 5 400 тактов на вызов, 13 вызовов за кадр = 16 % кадрового периода на «в какой колонке точка». 3. get_row_collision_data: ряд разрешается один раз на весь перебор (было — get_tile на каждую колонку, 1 422 такта), грань идёт шагом TILE_SIZEX как в оригинале, wall_type таблицей вместо switch. Итог: check_collisions 60 888 -> 42 750, физика Кида 100 578 -> 67 518, синяя полоса ~60 % -> ~30 % кадрового периода. Профиль остатка и следующие цели (отрисовка Кида 47 %, process_trobs 21 %) — в TASKS_OPEN.md#draw-cost. tests-host: все 5 наборов прошли. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
@@ -79,16 +79,25 @@ for (short column = 9; column >= 0; --column) {
|
||||
Не взяли по одной причине: **десяти слотов нам не хватит**. Оригинал
|
||||
перебирает узкое окно вокруг Кида (от `col(char_x_left_coll) − 1` до
|
||||
`col(char_x_right_coll) + 2`), поэтому коллизии слотов у него практически
|
||||
не случаются. Мы намеренно считаем все четырнадцать колонок сразу
|
||||
(FIX_COLL_FLAGS из SDLPoP — чтобы не оставалось протухших ячеек), а при
|
||||
таком переборе колонки −2/−1 и 8/9 сядут в одни и те же слоты 8/9. То
|
||||
есть вариант A тянет за собой ещё и сужение окна перебора — то есть отказ
|
||||
от FIX_COLL_FLAGS, — и это заметно рискованнее точечного сдвига.
|
||||
не случаются. Мы держим четырнадцать колонок (−2…11) — при узком окне это
|
||||
не мешает, а вот в десять слотов колонки −2/−1 и 8/9 сядут поверх 8/9.
|
||||
|
||||
**Окно перебора с 2026-08-09 у нас такое же, как в оригинале** (было: все
|
||||
четырнадцать колонок каждый кадр). Признак годности слота при этом не
|
||||
массив номеров комнат, как у оригинала, а ГРАНИЦЫ окна — четыре байта,
|
||||
которые `move_coll_to_prev` переносит в `prev` вместе с флагами; сравнение
|
||||
идёт по пересечению двух окон. Очистки массивов нет вовсе, то есть это
|
||||
дешевле оригинала, а смысл тот же (у него слот вне окна помечен
|
||||
`row_coll_room = −1` и в цикл бампа не попадает). `check_chomped_flags`
|
||||
тоже ограничен окном — иначе протухшие слоты дали бы фантомный перемол.
|
||||
|
||||
### Чем платим
|
||||
|
||||
- Расхождение структур: если в будущем понадобится знать, из какой комнаты
|
||||
пришёл тайл конкретного слота, этого у нас нет — придётся идти в вариант A.
|
||||
- Границы окна надо переносить везде, где переносятся флаги: `pop_coll_shift`
|
||||
двигает и их, `move_coll_to_prev` снимает их в `prev`. Забыть один из
|
||||
переносов = молча потерять или, наоборот, разрешить лишний бамп.
|
||||
- Сдвиг работает только для чисто горизонтальных переходов на ровно 10
|
||||
колонок. Любая будущая диагональ/иная ширина комнаты его сломает молча.
|
||||
- `coll_last_row`: `pop_coll_invalidate` прячет прошлый ряд, чтобы
|
||||
|
||||
Reference in New Issue
Block a user