XML output insufficiently qualified
The xml output generated by the verilator --xmlonly command is insufficiently qualified when it comes to toplevel ports.
The attached patch addresses the issue but you need to check if it will break any of your regression tests.
#1 Updated by Wilson Snyder 13 days ago
- Status changed from New to AskedReporter
I'm fine with this concept but can you instead please update the patch to add a method to VDirection and AstVarType class similar to the existing ones there that is called by V3EmitXml? (Or perhaps verilogKwd() is sufficient.) That gets rid of the internal-peeking switch statement.
#7 Updated by Jonathan Kimmitt 9 days ago
- File xml.patch5 added
I realized too late that the patch previously submitted does not cover dumping port directions for modports. The attached fix seems to be rather simple unless you know of undesirable side-effects. Sorry for the confusion over the previous message, I did not realise your repository is not browsable.
#12 Updated by Jonathan Kimmitt 7 days ago
- File xml.patch7 added
That resolution didn't last long. It turns out AstVarXRef's are not qualified with the path of the interface that they belong to (stored in the m_dotted field). I definitely predict the rate of submission of patches will decline from now on.
Also available in: Atom