search

The-OpenROAD-Project / RePlAce

RePlAce global placement tool
BSD 3-Clause "New" or "Revised" License
215 stars 75 forks source link

DEF parser does not recognize NAMECASESENSITIVE #28

Closed RTimothyEdwards closed 5 years ago

RTimothyEdwards commented 5 years ago

RePlAce fails when the DEF file contains the statement "NAMECASESENSITIVE ON ;"

mgwoo commented 5 years ago

The NAMESCASESENSITIVE statement is only valid for LEF syntax. I was not able to find this statement on DEF inside LEF/DEF 5.8 language manual... (RePlAce uses LEF/DEF parser and LEF/DEF language manual in this link: https://projects.si2.org/openeda.si2.org/project/showfiles.php?group_id=84#p145)

Could you check this again?

RTimothyEdwards commented 5 years ago

I was surprised not to find it in the LEF/DEF version 5.7 reference, either. Apparently it was deprecated and removed from the spec. However, it is specified in LEF/DEF version 5.4 for both LEF and DEF files, where the manual states that it must appear in the first line after the VERSION statement. However, since the LEF/DEF version is declared at the top of the file, the NAMECASESENSITIVE keyword should be recognized as part of the 5.4 spec, and not generate an error (even if it is ignored).

I am aware, however, that qflow is declaring version 5.6 and then adding the NAMECASESENSITIVE statement, and since I have only a manual for 5.4 and 5.7, for all I know this might not be legal syntax for 5.6.

RTimothyEdwards commented 5 years ago

Okay, never mind. The keyword is "NAMESCASESENSITIVE" and I managed to spell it wrong. Once spelled correctly, your parser flags it as obsolete in version 5.6, thus answering my other question. So this is not an issue at all except for my own programming error, which I just fixed.