Project

General

Profile

[logo] 
 
Home
News
Activity
About/Contact
Major Tools
  Dinotrace
  Verilator
  Verilog-mode
  Verilog-Perl
Other Tools
  IPC::Locker
  Parallel::Forker
  Voneline
General Info
  Papers

Issue #1499

Have V3EmitC::emitcSyms honor --output-split-cfuncs

Added by Todd Strader 3 months ago. Updated about 1 month ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Configure/Make/Compiling
% Done:

0%


Description

I'm planning to modify V3EmitC::emitcSyms to respect --output-split-cfuncs in order to break up the symbol object constructor. This looks reasonably straightforward with the existing class. However, I know the long term desire is to refactor this into a proper AST visitor class.

Would it be acceptable to add this feature to the existing class without refactoring it?

History

#2 Updated by Todd Strader 3 months ago

It just dawned on me that I didn't modify t_flag_csplit.pl to verify that it is now producing Syms__#.cpp files (which it is). I'll add that when I can get back to this.

#3 Updated by Wilson Snyder 3 months ago

  • Category set to Configure/Make/Compiling
  • Status changed from New to Assigned

Good stuff, thanks.

Once you update tests feel free to squash and push.

#4 Updated by Todd Strader 3 months ago

  • Status changed from Assigned to Resolved

Done. This has been pushed to git now.

#5 Updated by Wilson Snyder about 1 month ago

  • Status changed from Resolved to Closed

In 4.020. Thanks for reporting this; if there are additional related problems, please open a new issue.

Also available in: Atom