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.

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

Odb: split LefDefParser namespace into LefParser and DefParser #6396

Open
wants to merge 135 commits into
base: master
Choose a base branch
from

Conversation

LucasYuki
Copy link
Contributor

No description provided.

AcKoucher and others added 30 commits November 19, 2024 17:16
Signed-off-by: luis201420 <[email protected]>
Signed-off-by: luis201420 <[email protected]>
Signed-off-by: luis201420 <[email protected]>
Signed-off-by: osamahammad21 <[email protected]>
Signed-off-by: João Mai <[email protected]>
Signed-off-by: João Mai <[email protected]>
eder-matheus and others added 5 commits December 26, 2024 15:42
Signed-off-by: Eder Monteiro <[email protected]>
…ite_spef

rsz: add corner prefix when design has more than one corner
Signed-off-by: Lucas Yuki Imamura <[email protected]>
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang-tidy made some suggestions

src/odb/src/defin/definReader.h Show resolved Hide resolved
src/odb/src/defin/definReader.h Show resolved Hide resolved
src/odb/src/defin/definReader.h Show resolved Hide resolved
LucasYuki and others added 2 commits December 26, 2024 20:25
Signed-off-by: Lucas Yuki Imamura <[email protected]>
…t-staging/odb-rm-defines

odb: rm some #defines from lef/def parser
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

clang-tidy made some suggestions

src/odb/src/defin/definReader.h Show resolved Hide resolved
src/odb/src/defin/definReader.h Show resolved Hide resolved
src/odb/src/defin/definReader.h Show resolved Hide resolved
@maliberty
Copy link
Member

It seems a mechanical change so I didn't review it line by line. There is one CI failure to resolve.

LucasYuki and others added 19 commits December 26, 2024 21:18
Signed-off-by: Lucas Yuki Imamura <[email protected]>
Signed-off-by: Lucas Yuki Imamura <[email protected]>
Signed-off-by: Matt Liberty <[email protected]>
…t-staging/odb-modernize-use-using

odb: fix modernize-use-using
…t-staging/all-modernize-use-using

All modernize use using
Signed-off-by: Peter Gadfort <[email protected]>
odb: add mask information to tracks from DEF
…t-staging/clang-tidy-cast-and-inline

dpl/dpo/odb/utl: rm unnecessary casts and inlines
Signed-off-by: Lucas Yuki Imamura <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.