lowRISC / lowRISC/opentitan

[otbn] Add named "enums" for flag indices in assembly

Open
#4,473 1 comment 0 reactions 0 assignees View on GitHub
Component:Tooling Earlgrey-PROD Candidate IP:otbn Priority:P2 Type:FutureRelease Type:Task
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

These would have avoided one of the problems that Felix had to fix in PR #4459 (and generally should make the code easier to read). The easiest way to do it is probably for `otbn-as` to generate some `.equ` lines at the top of the code it passes to the risc-v assembler.

Contributor guide

Open the contributing guide

Research direction

Start by tracing the otbn-as path that passes assembly to the RISC-V assembler. Check how the generated assembly begins and determine where named .equ lines for flag indices should be emitted. Done means the generated code exposes readable named constants that avoid numeric flag indices and preserves existing assembly behavior.

Written by the indexing model from the issue text.

Assessment

Domain
compilers, tooling
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.