Re: [sv-ec] Question Regrading the Open value range list with Covergroup bins

From: Neil Korpusik <Neil.Korpusik_at_.....>
Date: Mon Apr 07 2008 - 17:26:53 PDT
<forwarding bounced email from Saurabh Sharma>


-------- Original Message --------
Subject: Re: [sv-ec] Question Regrading the Open value range list with Covergroup bins
Date: Mon, 07 Apr 2008 13:58:23 +0530
From: Saurabh Sharma <saurabhs@cadence.com>
To: Steven Sharp <sharp@cadence.com>, sv-ec@eda.org
CC: Neil.Korpusik@Sun.COM, saurabhs@cadence.com
References: <200804042351.m34Npvw11059@quicksand.Cadence.COM>

Hi Steven,

Thanks for your comments. I complete agree with you.

The only issue why i bought this in to notice is that
the BNF may mislead users to create erroneous scenario as shown below

module example;
    logic clk;
    int tmp;

   covergroup cg2 @(posedge clk);
    ce: coverpoint var1 {
                bins  b = { [tmp : 100]
};
                 }
        endgroup :cg2

        cg2 cg2_inst = new;

endmodule

This  of course is against what is expected by the LRM.

Thanks
Saurabh Sharma


Steven Sharp wrote:

>>         " The open_range_list used to specify the set of values associated 
>>    
>>
>with
>  
>
>>           a bin shall be *constant* *expressions* (see 11.2.1), instance 
>>    
>>
>constants
>  
>
>>           (for classes only), or non-ref arguments to the coverage group."
>>
>>        But the  BNF is as shown below.
>>
>>       open_range_list ::= open_value_range { , open_value_range }
>>       open_value_range ::= value_range
>>       value_range ::= expression | [ expression : expression ]
>>
>>      I feel that here it should be "constant_expression" and not "expression"
>>      and should be corrected.
>>    
>>
>
>If the BNF only allowed constant_expressions, then the language would
>only allow constant expressions.  But from the text you quoted, it allows
>instance constants for classes, and non-ref arguments to the coverage group.
>To allow these, the BNF must allow them.
>
>It might be possible to restrict this a little more with the BNF, but it
>is simpler to use the general production and restrict it with the semantic
>rules you quoted.
>
>Steven Sharp
>sharp@cadence.com
>
>  
>


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.
Received on Mon Apr 7 17:27:25 2008

This archive was generated by hypermail 2.1.8 : Mon Apr 07 2008 - 17:28:34 PDT