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.

Sign up for GitHub

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

Jump to bottom

Converter from JSON to Liberty #1

Closed
wants to merge 39 commits into from
Closed

Converter from JSON to Liberty #1

wants to merge 39 commits into from

Conversation

glatosinski
Copy link
Member

This MR:

  • adds more import features from Liberty to JSON
  • adds converter from JSON to Liberty
  • adds script for testing the conversion process
  • adds a tool for "fuzzy" comparison of Liberty files

@glatosinski
Removed parsing header and postprocessing JSON from parsing function
41dc6a4 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Extracted variable and indent regular expressions
ace0283 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added removing Python-like comments
0fa3a7b 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Perform all line processing in one loop
b03a222 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added inserting semicolon for lines that do not have it
5a078ab 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added parsing define entries
a32504f 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added parsing library attributes
97cb2f6 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added parsing numeric arrays for nonlinear timings
bfbb7b1 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added removing date entries from file
06892fb 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added Liberty-to-JSON converter
7fb039a 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added .gitignore
00a47f7 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Separated log functions and Liberty-to-JSON functions from quicklogic…
9fb5a6e 
…_timings_importer

Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Fixed parsing for float values, fixed parsing attributes
64d0d5a 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added complex attribute indicator
22c5962 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added space to indicate empty names for structs
37d1ba8 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added support for new timing form
3170354 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added initial version of converter from JSON to Liberty
0bd914b 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added script for verifying LIB-JSON-LIB-JSON pipeline
da24a5a 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Fixed index_* entries
5c6995d 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Fixed 2D array declaration - removed comma from last line
1b31e32 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Skipped wrapping true/false values in quotes
b5b1915 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Improved quote handling
adca776 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added tool for "fuzzy" comparing of Liberty files
02bab99 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Fixed extracting complex attribute type
01b0607 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added duplicate removal for Liberty-to-JSON parser
aad1bd9 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Removed date removal from Liberty files
c37b478 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added testing similarity between Liberty files
d13605b 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Cleaned lib_diff
ab2f294 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Cleaned test_full_pipeline
bc607d2 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Cleaned json_to_liberty
50b9c25 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Cleaned liberty_to_json
4d9c03a 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Documented lib_diff methods
5ad1c30 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added possibility to save log to file
29d23c0 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski glatosinski assigned kgugala and unassigned kgugala Mar 23, 2020
@glatosinski
Removed adding entries with no values in SDF converter
14a883c 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski glatosinski changed the title Converter from JSON to Liberty [WIP] Converter from JSON to Liberty Mar 23, 2020
@glatosinski
Fixed error with returning 'pin ' in SDF file
c7bd7a1 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Removed conditional delval removal in setup/hold entries
7c93e5f 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski glatosinski changed the title [WIP] Converter from JSON to Liberty Converter from JSON to Liberty Mar 24, 2020
@kgugala
Copy link
Member

kgugala commented Mar 25, 2020

can you rebase this PR (it has conflicts with master)

@glatosinski
Fixed parsing C++ comments
10aca71 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@glatosinski
Added returning non-zero exit code upon invalid tests
9efb48e 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@kgugala kgugala mentioned this pull request Apr 29, 2020
Open
@glatosinski
Removed creating debug file
22b0e34 
Signed-off-by: Grzegorz Latosinski <glatosinski@antmicro.com>
@kgugala
Copy link
Member

kgugala commented May 6, 2020

this code has been merged manually. Closing this PR

@kgugala kgugala closed this May 6, 2020
acomodi pushed a commit that referenced this pull request Aug 12, 2020
@sharmaln
Merge pull request #1 from antmicro/master
481703d 
Optional cells and port names normalization
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kgugala kgugala Awaiting requested review from kgugala

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@glatosinski @kgugala