Matt Liberty comments

Results 1716 comments of


                                            Matt Liberty

Right click in timing report, go to verilog line

You should note that we will navigate back to the always statement. If you put a wrapper around your register then all instances will map to the same line. It...

Right click in timing report, go to verilog line

https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/pull/2043 enables src attributes in yosys towards this end.

Right click in timing report, go to verilog line

#5216 converts verilog attributes to odb ones to enable the GUI work

Right click in timing report, go to verilog line

I've done preparatory work in ORFS & odb (see PRs above) so it should just be a GUI issue now.

Right click in timing report, go to verilog line

Some googling suggests `QDesktopServices::openUrl(QUrl::fromLocalFile(""));` but I haven't actually tried it. Getting it to go to a specific line is likely very application specific and won't work through such a general...

Right click in timing report, go to verilog line

yeah not really a solution

Right click in timing report, go to verilog line

How about something like a user envar to configure the viewer. For example `setenv VIEWER "vscode -goto :"` where we sub `` and `` at runtime Then you could use...

Right click in timing report, go to verilog line

It seems simpler to just open it in a Qt window. This seems like a lot of extra complication and will generate lots of user confusion.

Right click in timing report, go to verilog line

Where would the user go to set it? Putting it in the timing report settings seems odd (also the inspector would want it) but we don't have a general settings...

Right click in timing report, go to verilog line

@oharboe I think it might be more generally useful if a right click went to the inspector for the instance and the inspector had line info that could be clicked...