verilator icon indicating copy to clipboard operation
verilator copied to clipboard

Published 20 hours ago • verified publisher icon verilator

Improve internals of inliner

Open veripoolbot opened this issue 6 years ago • 2 comments

Author Name: Wilson Snyder (@wsnyder) Original Redmine Issue: 1539 from https://www.veripool.org

Feature tracking bug.

The current module inliner code works by operating on modules before the complete scoped hierarchy has been created (V3Scope).

This should be changed to be post-scope, as this would also remove the horrible need to go through V3LinkDot for the final time to cleanup inlined references. This V3LinkDot pass is very difficult to maintain.

Some work was started on this about a year ago and is incomplete.

veripoolbot avatar Oct 06 '19 12:10 veripoolbot

@wsnyder is this still expected to happen? I was planning to make some improvements in the current inline pass, but this makes me wonder if it's futile.

gezalore avatar Dec 21 '21 15:12 gezalore

This stalled out in a branch as interfaces proved a big mess (as usual). If you have improvements, perhaps short of a big rewrite, feel free to put them in, and if/when I get back to this I'll port the changes to the new version. If you're interested in doing something major I can put what I have onto a branch.

wsnyder avatar Dec 21 '21 16:12 wsnyder