-
Notifications
You must be signed in to change notification settings - Fork 745
odb: add 3dbxWriter #8775
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
base: master
Are you sure you want to change the base?
odb: add 3dbxWriter #8775
Conversation
There was a problem hiding this 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
Signed-off-by: Rafael Moresco <[email protected]>
Signed-off-by: Rafael Moresco <[email protected]>
b35831b to
5487322
Compare
Signed-off-by: Rafael Moresco <[email protected]>
There was a problem hiding this 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
| const double loc_x = inst->getLoc().x() / u; | ||
| const double loc_y = inst->getLoc().y() / u; | ||
| YAML::Node loc_out; | ||
| loc_out.SetStyle(YAML::EmitterStyle::Flow); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: no header providing "YAML::EmitterStyle" is directly included [misc-include-cleaner]
src/odb/src/3dblox/dbxWriter.cpp:5:
- #include <yaml-cpp/yaml.h>
+ #include <yaml-cpp/emitterstyle.h>
+ #include <yaml-cpp/yaml.h>
Add 3dbxWriter.
Still not connected to main code, first commit is for analysis only. 3dbvWriter is needed for proper working. CLI would call the writeFile function, where the user gives the filename. This function will call the 3dbv writer for the top chiplet, writting all the dependencies in a file. In the case of a hier chiplet, 3dbvWriter will call the writeChiplet function, that has the same internal working but creates its own file name.