habibayassin

Results 13 comments of habibayassin

> `getWidth()` and `getMinWidth()` can also be replaced. @rovinski I actually tried to replace them, but they are being set with values other than the ones from dbtechlayer->getWidth() or dbtechlayer->getMinWidth()

> How so? It looks like they are: > > https://github.com/The-OpenROAD-Project/OpenROAD/blob/a2b9a98b5917fddaa9e99d801328f3f8d0e9d039/src/drt/src/io/io.cpp#L1521-L1528 @rovinski it is being set here as well: https://github.com/The-OpenROAD-Project/OpenROAD/blob/a2b9a98b5917fddaa9e99d801328f3f8d0e9d039/src/drt/src/io/io_parser_helper.cpp#L286

> The build still failed as shown in the log below. > > [build_openroad.log](https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/files/10445657/build_openroad.log) @vvbandeira I opened [this PR](https://github.com/The-OpenROAD-Project/OpenROAD/pull/2798) in OpenROAD as the above log file is generating an error...

> @vvbandeira Can you update `OpenROAD` submodule as well to reflect the changes? @vvbandeira @vijayank88 I have an open [PR](https://github.com/The-OpenROAD-Project/OpenROAD-flow-scripts/pull/697) that has the updated OR submodule ready to be merged

> @habibayassin For MacOS unable to create `/opt/or-tools` currently: > > ``` > ./etc/DependencyInstaller.sh -local > > mkdir: /opt/or-tools: Permission denied > ``` > > If use `sudo`: > >...

final def output before: > VERSION 5.8 ; DIVIDERCHAR "/" ; BUSBITCHARS "[]" ; DESIGN uart ; UNITS DISTANCE MICRONS 1000 ; DIEAREA ( 0 0 ) ( 17000 17000...

final def after mangling: > VERSION 5.8 ; DIVIDERCHAR "/" ; BUSBITCHARS "[]" ; DESIGN uart ; UNITS DISTANCE MICRONS 1000 ; DIEAREA ( 0 0 ) ( 17000 17000...

[reduced_asap7_lib_and_lef.zip](https://github.com/user-attachments/files/15780010/reduced_asap7_lib_and_lef.zip)

@oharboe do you think we need to mangle the name of the PDK from the final def file?