TopLCS2016_I15 < P1076

P1076 Web>IssueScreening>AlternatePathName>TopLCS2016_I15 (2017-03-21, LievenLemiengre) (raw view)EditAttach
---+ Language Change Specification for Generate Statement Alternate Path Names Proposal

<table border="1" cellpadding="0" cellspacing="0" id="table1" rules="all"> <tbody> <tr> <td bgcolor="#ffffff" valign="top"> *LCS Number:* </td> <td bgcolor="#ffffff" valign="top">LCS-2016-I15</td> </tr> <tr> <td bgcolor="#edf4f9" valign="top"> *Version:* </td> <td bgcolor="#edf4f9" valign="top">1</td> </tr> <tr> <td bgcolor="#ffffff" valign="top"> *Date:* </td> <td bgcolor="#ffffff" valign="top">Dec-1-2016</td> </tr> <tr> <td bgcolor="#edf4f9" valign="top"> *Status:* </td> <td bgcolor="#edf4f9" valign="top"> </td> </tr> <tr> <td bgcolor="#ffffff" valign="top"> *Author:* </td> <td bgcolor="#ffffff" valign="top">Kevin Jennings</td> </tr> <tr> <td bgcolor="#edf4f9" valign="top"> *Email:* </td> <td bgcolor="#edf4f9" valign="top"><a href="http://www.eda-twiki.org/cgi-bin/view.cgi/Main/KevinJennings" target="_top">KevinJennings</a></td> </tr> <tr> <td bgcolor="#ffffff" valign="top"> *Source Doc:* </td> <td bgcolor="#ffffff" valign="top"> [[AlternatePathName][Generate Statement Alternate Path Names]]</td> </tr> <tr> <td bgcolor="#edf4f9" valign="top"> *Summary:* </td> <td bgcolor="#edf4f9" valign="top">Generate Statement Alternate Path Names</td> </tr> </tbody> </table>
---+++ Voting Results: Cast your votes here

Yes:
   1 %USERSIG{PatrickLehmann - 2016-12-09}%
   1 %USERSIG{MartinThompson - 2016-12-12}%

No:
   1 %USERSIG{JimLewis - 2017-02-25}% -- see note. There is no alternative label for case generate or if generate in the EBNF for instance_name and path_name, so it is already clear in the LRM. Hence, I don't even think a note is warranted. OTOH, maybe we want these in instance_name and path_name and hence, see [[TopLCS2016_I15a][LCS_2016_015a]] - it is ok to vote no on both<br /> <p> </p> <p> </p> <p> </p>

Abstain:
   1 %USERSIG{KevinJennings - 2016-12-20}% - ver 1
   1 %USERSIG{MartinZabel - 2017-01-30}% Any version - I have no experience with external names.
   1 %USERSIG{BrentHahoe - 2017-02-16}% Version 1 - Abstain due to lack of personal time for review. <p> </p>
   1 %USERSIG{MartinThompson - 2017-02-17}% Version 1
   1 %USERSIG{LievenLemiengre - 2017-03-21}% Version 1
---+++ Details of Language Change:

Key:
   * Existing LRM text is shown in BLACK font
   * Additional LRM text is shown in <span style="background-color: transparent;">RED font</span>
   * Deleted LRM text is shown in <span style="background-color: transparent;"><del>RED with strike-through</del></span> <p> </p>
---++++ LRM 8.7 bullet b)5) page 115 near top

Create new sub-bullet c) after sub-bullet b)5) adding the following text:

<span style="background-color: transparent;">c) </span><span style="background-color: transparent;">The alternative label possible in an 'if generate' or a 'case generate' statement does not form part of the path name.</span>

---++ Comments

Adding a c) doesn't make sense with the surrounding text.How about modifying (b.5) directly:

"5) For a generate statement label, the declarative region of the equivalent block corresponding to the generate statement.<span style="background-color: transparent;">The alternative label possible in an 'if generate' or a 'case generate' statement must not appear in the path name.</span>If the generate...."

-- <span style="background-color: transparent;"><span style="background-color: transparent;">%BUBBLESIG{RyanHinton - 2016-12-20}%</span></span>

<span style="background-color: transparent;"><span style="background-color: transparent;"><br /></span></span>

<span style="background-color: transparent;"><span style="background-color: transparent;"><span style="background-color: transparent;">
First I disagree with the bugzilla poster's assumption that the language did not intend to have those as part of the path name.  </span></span></span>

<span style="background-color: transparent;"><span style="background-color: transparent;">
Both path name and instance name also include things more trivial things like loop labels.   In addition, a "for generate" loop index value is included as part of the generate statement label in both path name and instance name.</span></span>

<span style="background-color: transparent;"><span style="background-color: transparent;">
Also whether you put a not somewhere else, this needs to be fixed in section 16.2.5  under path name and instance name.   Currently the alternate labels are not included in the syntax - they need to be added.  Like process label and loop label, if the optional label is not specified, it is empty label is included.   Empty label has no characters.  Hence, a path with an empty label has two consecutive path separators.</span></span>

<span style="background-color: transparent;">
-- %BUBBLESIG{JimLewis - 2017-02-25}%
%COMMENT%</span>
Topic revision: r15 - 2017-03-21 - 17:46:35 - LievenLemiengre
P1076