Making printing "// Templated" an option?

[aolofsson]

I make extensive use of AUTOINST for stitching together designs and I do find that the static comments at the end of each instance port add little value, except on rare occasions as a quick filter to see if I messed up a templates. Very few of the next generation of designers use emacs (it's pretty much 100% vs code in my team) and they find that the "// Templated" clutter up the code. My goal is always write once, read many times so making the code cleaner looking is important.

I know this is probably low on the priority list, but if there is a way to enable something like what we have elsewhere, I sure would appreciate it. I suspect others might like it as well.

eg. (setq verilog-auto-endcomments nil)

                 .RXRDY                (NRXRDY[i]),             // Templated
                  .TXCKP                (NTXCKP[i]),             // Templated
                  .TXCKN                (NTXCKN[i]),             // Templated
                  .TXVLD                (NTXVLD[i]),             // Templated
                  .TXD                  (NTXD[i*IOW+:IOW]),      // Templated
                  .TXTRK                (NTXTRK[i]),             // Templated
                  .TXCKSB               (NTXCKSB[i]),            // Templated
                  .TXDSB                (NTXDSB[i*IOW+:IOW]),    // Templated
                  .io_txck              (no_txck[i]),            // Templated
                  .io_txvld             (no_txvld[i]),           // Templated

Previous issue:

#1764

[wsnyder]

Hi Andreas! There's already verilog-auto-inst-template-numbers we could add 'none to that. Might you be willing to make a pull for this including a test?

[aolofsson]

Thanks Wilson! I will give it a shot.

[furkanu-work]

As a +1 to this option I have this code

module sub #(parameter PARAM=3) ();
endmodule

module top();
     /* sub AUTO_TEMPLATE ( .PARAM(5), ); */
    sub #(/*AUTOINSTPARAM*/) sub_i (/*AUTOINST*/);
endmodule

When expanded instance name stays at the same line as the last "// Templated" comment and gets commented out

    sub #(/*AUTOINSTPARAM*/
          // Parameters
          .PARAM                        (5))                     // Templated sub_i (/*AUTOINST*/);