Skip to content

fix: align Shared Tile PE mask ordering - #8

Merged
zhoubot merged 1 commit into
mainfrom
codex/v058-pe-mask
Aug 11, 2026
Merged

zhoubot merged 1 commit into
mainfrom
codex/v058-pe-mask

Conversation

@zhoubot

@zhoubot zhoubot commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Summary

  • map architectural Shared Tile PE masks as bit3=PE0 through bit0=PE3
  • update the differential fixture and its fail-closed validator
  • remove version-number suffixes from C++ test/helper names

TDD evidence

  • old implementation failed the updated state test with emulator_rc=32
  • old fixture failed with PE mask must map bit3=PE0 through bit0=PE3

Verification

  • complete build succeeded
  • ctest --test-dir build-v058 --output-on-failure: 10/10 passed
  • clang-format check passed
  • forbidden C++ version-name scan returned no matches
  • local markdownlint unavailable; hosted workflow remains authoritative

Corrects the model portion of LinxISA/linx-isa#165 and aligns with LinxISA/LinxCore#48.

Signed-off-by: RuoyuZhou <ruoyu.zhou@hisilicon.com>
@zhoubot
zhoubot merged commit 69e926b into main Aug 11, 2026
10 checks passed
@zhoubot
zhoubot deleted the codex/v058-pe-mask branch August 11, 2026 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant