Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

about annotation #7

Open
KingFrige opened this issue Oct 21, 2019 · 2 comments
Open

about annotation #7

KingFrige opened this issue Oct 21, 2019 · 2 comments

Comments

@KingFrige
Copy link

KingFrige commented Oct 21, 2019

wujian100_open 30/105029*100%=0.03%

$ find . -type f | xargs cat | wc -l
105029
$ grep -rn "//" |wc -l
30
$ pwd
~/wujian100_open/soc

$ grep -rn "//" 
sms.v:466:casex ({resp_cfg[3:2], rty_first, resp_cfg[1:0]})  // synopsys parallel_case
sms.v:555:  case (ram_size[2:0])   // synopsys parallel_case
sms.v:557:      case (ram_addr[1:0])   // synopsys parallel_case
sms.v:568:      case (ram_addr[1])     // synopsys parallel_case
common.v:28:  case(clk_sel) // synopsys infer_mux
E902_20191018.v:194:  //  `define HAD_MBKPT_9 
E902_20191018.v:197:    //`define DBG_EXP 
usi0.v:1066:                case({i2cm_hs, i2cm_sbyte}) // synopsys parallel_case
usi0.v:2839:        case(txcnt_bits[4:1]) // synopsys parallel_case
usi0.v:2866:    case(spi_data_size[3:0]) // synopsys parallel_case
usi0.v:2916:        case(txcnt_bits[4:1]) // synopsys parallel_case
usi0.v:2981:        case(cnt_bits[3:0]) // synopsys parallel_case
usi0.v:3071:    case(spi_data_size[3:0]) // synopsys parallel_case
usi0.v:3555:                case(txcnt_bits[2:0]) // synopsys parallel_case
usi0.v:3581:    case(uart_dbit[1:0])  // synopsys parallel_case
usi0.v:3616:        case(uart_pbit[1:0]) // synopsys parallel_case
usi0.v:3669:    case(uart_dbit[1:0]) // synopsys parallel_case
matrix.v:224:// synopsys translate_off
matrix.v:237:// synopsys translate_on
matrix.v:452:// synopsys translate_off
matrix.v:465:// synopsys translate_on
dmac.v:933:    case(data_chnl_cd)  // synopsys parallel_case
dmac.v:957:        //data_chnl_cd[3:0] <= 4'b00 ; 
dmac.v:965:    case(busy_chn_code)  // synopsys parallel_case
dmac.v:986:    case(busy_chn_code)  // synopsys parallel_case
dmac.v:1007:    case(busy_chn_code)  // synopsys parallel_case
dmac.v:1029:    case(busy_chn_code)  // synopsys parallel_case
dmac.v:15707:assign   cntr_blk_decen = ( is_wr_ctrlstt ) & current_rcvvld & (~blk_nu_eql0) ; //& is_blk_src |  
dmac.v:15808:                               //src_single_inten & is_rd_ctrlstt & current_rcvvld   |     
dmac.v:15809:                               //dst_single_inten & is_wr_ctrlstt & current_rcvvld  

rocketchip 3256/45594*100% = 7.14%

$ find . -type f | xargs cat | wc -l
45594
$ grep -rn "//" |wc -l
3256
$ pwd
~/rocket-chip/src

e203 6797/77766*100%=8.74%

$ find . -type f | xargs cat | wc -l
77766
$ grep -rn "//" |wc -l
6797
$ pwd
~/e200_opensource/rtl/e203
@diodep
Copy link

diodep commented Oct 22, 2019

In fact it seems they deleted all comments by script.

Most of comments you have listed are synthesis hint. I haven't seen any real comments (I mean make sense for human) in this code.

So I would like to say this code have zero comment.

@troyguo
Copy link

troyguo commented Oct 22, 2019

It seems they delete the comments intently. What's more, some parts of the core are encrypted...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants