llvm / llvm/circt

[exportverilog] Improve location options

Open
#899 5 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

We currently print source location information as a comment with line and column info. It would be great to expand this (along with a lowering option) to four different options:

1) None
2) Current comment.
3) `line directives (which cover filename + line)
4) `line and comment (which covers filename+line+col)

Something like `veriloglocationstyle={none,comment,line,both}` would be nice.

-Chris

Contributor guide

No contributing guide indexed for this repository

Research direction

The issue targets the exportverilog lowering and its source-location output; start by locating the existing comment emission and lowering-option handling. Done means the lowering exposes none, comment, line, and both styles, with the requested filename, line, and column coverage.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
compilers
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.