#3 Patch to support some VHDL 2008 Features
VHDL 2008 Feature support:
(IEEE1076-2008)
Section 11.8 Generate Statements
- supporting "elsif generate" and "else generate" for if_generate_statement and added case_generate_statement
- This feature is also cover by the patch in https://sourceforge.net/p/veditor/tickets/1/
Section 4.7 Package declaration
- Generic Package support
Section 4.9 Package instatntiation
- supporting instantiation of generic packages
Section 15.8 Bit string literals
- updated definition of bit_string_literals, bit_value and base_specifier
Section 15.2 Character set
- updated character set support
Section 8.7 External names
- support access to objects in the design hierarchy
Section 9.2.3 Relational operators
- adding support for Matching operators
Additionally I fixed a trifle:
Currently the parser reports an error when identifiers are the same but differ in their case (lower/upper case) (e.g. "identifiers don't match: Rtl /= RTL"). Maybe it's not a good style to mix upper and lower case but it is not a VHDL error because VHDL is case insensitive.
For more details have a look at IEEE1076-2008 Section 15.4.2 which says "Basic identifiers differing only in the use of corresponding uppercase and lowercase letters are considered the same."
I changed the "compareTo" function call to "compareToIgnoreCase" in ASTarchitecture_body.java, ASTentity_declaration.java and ASTprocess_statement.java
Hi,
just wondering why you do not integrate this patch in your releases...
I'm considering to fork this project to get more VHDL 2008 support integrated.
Sorry about the delayed response. I'm swamped with work and Tadashi, (the
other developer) is mainly interested in Verilog changes. Would you like me
to add you as a developer so you can graft your patches directly?
-Ali
On Fri, Feb 19, 2016 at 6:31 AM, HorstWe horstwe123@users.sf.net wrote:
--
-Ali
Related
Tickets: #3
Yes, you can add me as a developer. I will integrate those changes and very lickely adding some other 2008 language improvements to the VHDL parser.
Done
Any updates here on integrating this patch?
Hi Ruben, unfortunately I haven't had time yet, but i it is planned to do it in the next few weeks.
But in the meanwhile I attached an older version of my local build (1.2.0) (latest is 1.2.1) which has the patch integrated.
Hi,
I applied the VHDL-2008-Features.patch and commited to SVN, except two lines.
The patch file includes non ascii code characters at line 147 and 148. They are 0xc0, 0xdf, 0xe0 and 0xff.
I deleted lines from the patch. So upper and lower case letter definitions remain these:
<#upper_case_letter: ["A"-"Z"]>
<#lower_case_letter: ["a"-"z"]>
Hi Tadashi,
thank you for applying the patch.
Those characters are from the ISO/IEC 8859-1:1998 set as described VHDL 2008 Standard chapter "15.2 Character set". They compiled and worked without trouble on my system, but if there is a problem on your system I'm fine with it when we remain those on the old definition.
Hi,
Thank you for your information.
I didn't know ISO-8859-1. Now I can see ISO-8859-1 characters on my own Eclipse when I change "Text file encoding".
But unfortunately my installed JavaCC 6.0 shows errors. I will explore to fix the promblem.
BTW:
I posted version 1.2.2 to "veditor runtime". But it does not include your patches. It is same with development build 1.2.1.c. I will post new development build 1.2.2.a with your patches and my own updates soon.