wegamekinglc / Derivatives-Algorithms-Lib
82%
master: 85%

LAST BUILD BRANCH: codex/fix-msvc-utf8-charset
DEFAULT BRANCH: master
Repo Added
Token RCHt5bHBHiesAZrXN9ctLtgBNi9sbYmV8 regen
Build 3709 Last
Files 222
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH fix/script-ifelse-bstack-underflow
branch: fix/script-ifelse-bstack-underflow
CHANGE BRANCH
x
Sync Branches

coverage: 82.244% (+0.2%) from 81.996%
28700916027

Pull #204

github

wegamekinglc 
Fix IfElse bStack underflow in compiled evaluator

EvalCompiled reset the thread_local bStack on every entry, including the
recursive call the IfElse case made to evaluate the true branch. That
re-entry wiped the parent frame's bStack (where the just-tested condition
still sat), so the parent's subsequent bStack.Pop() underflowed sp_ and a
later Push wrote into the call frame's locals. Every IfElse whose
condition was true triggered it; Phase 0's UBSAN run confirmed the
diagnostic (stacks.hpp:133,135 'index -1 out of bounds for type bool [128]').

Gate the Reset() calls on a reset param: EvalCompiled keeps the opcode
switch with the master shape (one function, thread_local static stacks
inline), but reset defaults true and is only passed false by the IfElse
true-branch re-entry over (i+3, nodeStream[i+1]). The per-event entry
(event.hpp:136) uses the defaults and resets as before; the re-entrant
call preserves the parent's stacks.

Enabled the two Phase-0 IfElse parity pins
(TestParity_IfElse_ConsecutiveBothTrue, TestParity_IfElse_NestedInTrueBranch)
which go from UBSAN-crash to GREEN. The nested case exercises the
re-entrancy the reset=false path must support.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Pull Request #204: fix: IfElse bStack underflow in compiled evaluator

4 of 4 new or added lines in 1 file covered. (100.0%)

6429 of 7817 relevant lines covered (82.24%)

3319139.63 hits per line

Relevant lines Covered
7817 RELEVANT LINES 6429 COVERED LINES
3319139.63 HITS PER LINE
Source Files on fix/script-ifelse-bstack-underflow

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
28700916027 fix/script-ifelse-bstack-underflow Fix IfElse bStack underflow in compiled evaluator EvalCompiled reset the thread_local bStack on every entry, including the recursive call the IfElse case made to evaluate the true branch. That re-entry wiped the parent frame's bStack (where the j... Pull #204 wegamekinglc github
82.24
28700622085 fix/script-ifelse-bstack-underflow Fix IfElse bStack underflow in compiled evaluator EvalCompiled reset the thread_local bStack on every entry, including the recursive call the IfElse case made to evaluate the true branch. That re-entry wiped the parent frame's bStack (where the j... Pull #204 wegamekinglc github
82.24
See All Builds (2285)

Badge your Repo: Derivatives-Algorithms-Lib

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh