llvm / llvm/circt

[ExportVerilog] Add option to limit identifier length

Open
#10,356 4 comments 0 reactions 0 assignees View on GitHub
ExportVerilog
Dominant language
C++
Stars
2.2k
Forks
524
Avg merge
3d 2h
Merged PRs (30d)
46

Description

Some frontends emit reaaaaaaaally long identifiers. Not a problem for CIRCT but some EDA tools (*cough* *cough* Quartus) choke on those long identifiers. We should add an option to limit the number of characters in identifiers we emit to verilog to `N`.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the ExportVerilog entry point and the code that emits identifiers. Trace how an option is defined and passed into identifier emission, then verify that generated Verilog identifiers are limited to N characters using the existing ExportVerilog tests or an appropriate focused test.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
compilers
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.