llvm / llvm/circt

[SV] Rename "sym_name" in GenerateOp to something different

Open Beginner friendly
#11,079 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
2.2k
Forks
524
Avg merge
3d 2h
Merged PRs (30d)
46

Description

GenerateOp won't have symbol/SymbolOpInterface so "sym_name" is simply confusing. Not sure a right name but "name", "label" might be good candidates.

Contributor guide

No contributing guide indexed for this repository

Research direction

Locate GenerateOp and all references to its sym_name field or argument, then review the surrounding operation definition to choose a name that does not imply SymbolOpInterface support. Rename the field consistently and verify that no sym_name references remain for GenerateOp.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
compilers
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.