Krzysztof Bieganski

> > * This doesn't support waiting on events from class methods (which is something that does appear in UVM). That's due to the way triggers work in V5 –...

I've been doing some research on it, but it's not a priority. I wouldn't expect a usable implementation anytime soon. Contributions are welcome :)

This is now supported, see: https://github.com/verilator/verilator/pull/4626

@hzeller Pinging in case you missed it

@hzeller I looked at the other perprocessor PRs you mentioned, and I think most of it is in `master`, except for `+define+` handling (even though Verible [claims to support it](https://github.com/chipsalliance/verible/blob/529b51979a1e4ce9c14eca398a8209056243af37/verilog/tools/preprocessor/verilog_preprocessor.cc#L230-L231)?...

The biggest challenges right now, from looking at the tests, are: - Matching column alignment across preprocessor control flow. Unfortunately, while formatting a single variant of the file, the formatter...

The Windows and MacOS failures were basically due to some incorrect/UB pointer arithmetic. It works now. There's also the Read the Docs failure, not sure what that's about. The multi-pass...

> What is the use case? There are examples in the PR description. > Why can the existing XML format not be used instead? It affects verilation (disables stages, etc)....

* XML is pretty rare these days. JSON has a healthier ecosystem, at least in the open source space (e.g. tools like `jq`, various viewers, converters...) * One of the...

> 1. A file that is somewhat more stable than the AST, is stable run-to-run, but it is still coupled somewhat tightly to Verilator version. (e.g. --xml-only presently) > >...