diff --git a/MobileGL/MG_Backend/DirectVulkan/Renderer/WireDeclineTally.h b/MobileGL/MG_Backend/DirectVulkan/Renderer/WireDeclineTally.h index 74f868ec..3bed82a0 100644 --- a/MobileGL/MG_Backend/DirectVulkan/Renderer/WireDeclineTally.h +++ b/MobileGL/MG_Backend/DirectVulkan/Renderer/WireDeclineTally.h @@ -149,7 +149,9 @@ private: }; // Every decline goes through this: the tally moves ALWAYS, the log line is once per site. -// `return MGL_WIRE_DECLINE_FALSE(Site, "why", ...)` reads as the statement it replaces. +// `MGL_WIRE_DECLINE_AT(Site, "why", ...); return false;` is the statement pair that replaces +// a bare `return false;` (the few sites that call WireDeclineTally::Count directly stand +// under an MGLOG_W/E of their own; scripts/ci/wire_declines_audit.py holds them to that). // // A DECLINE IS NOT ALWAYS A SKIPPED DRAW, so the line does not say "draw". The Tex*, Shape* and // Upload* rows are counted per texture SYNC: SyncTextureResourceByHandle is reached from the diff --git a/MobileGL/MG_Backend/DirectVulkan/Renderer/WireDeclines.def b/MobileGL/MG_Backend/DirectVulkan/Renderer/WireDeclines.def index 13b9e0b8..fc5b8149 100644 --- a/MobileGL/MG_Backend/DirectVulkan/Renderer/WireDeclines.def +++ b/MobileGL/MG_Backend/DirectVulkan/Renderer/WireDeclines.def @@ -6,7 +6,13 @@ // SPDX-License-Identifier: LGPL-3.0-only // End of Source File Header -// P7 wave 2 package B3. THE NAMES OF EVERY WAY A MAGMA WIRE DRAW CAN DECLINE. +// P7 wave 2 package B3. THE NAMES OF EVERY WAY THE MAGMA WIRE ARM CAN DECLINE. +// +// A DECLINE IS NOT ALWAYS A SKIPPED DRAW. The SetupWireDraw / DispatchWireCompute rows are one +// per refused draw or dispatch; the Tex* / Shape* / Upload* rows are counted per texture SYNC, +// which the draw path reaches once per sampled texture and the descriptor resolver, framebuffer +// attachments, CopyTexSubImage, mipmap generation, CopyImageSubData and texture readback reach +// on their own (WireDeclineTally.h, above MGL_WIRE_DECLINE_AT). // // CONTRACT-P7 ยง0 rule I says a refusal on the wire arm is either a named decline or a named // Fatal. Before B3 the draw path had a third shape the rule does not admit: a bare