Codecov Report

Merging #573 (54c4f0f) into master (b466b72) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #573      +/-   ##
==========================================
- Coverage   81.50%   81.42%   -0.08%     
==========================================
  Files          49       49              
  Lines        6461     6461              
  Branches     1287     1287              
==========================================
- Hits         5266     5261       -5     
- Misses       1008     1009       +1     
- Partials      187      191       +4     

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b466b72...54c4f0f. Read the comment docs.

Using a regex for this purpose is completely unacceptable.

@whitequark can you give me a clue to where the right place would be for IL? I looked into it, but the src comments are all over the place in the code.

I don't think this is necessary to do for the IL at all.

I got it: in _ModuleBuilder.__enter__. I would like to actually read and understand the IL too betimes, and also there the source line comments really get in the way of readability

In that case you can use Yosys to remove the comments like it is done in back.verilog.

You mean, run yosys to filter in rtlil.py:convert or external to nmigen? Inferring from your previous comment, I suppose you suggest the latter.