OpenROAD drt: replace name/xform with dbInst* in frInst

Save a bit of memory and a small step towards odb integration.

Oct 25 '23 15:10 openroad-robot

@osamahammad21 does this have any effect on distributed routing? I don't see the frInst being serialized anywhere.

Oct 25 '23 15:10 maliberty

clang-tidy review says "All clean, LGTM! :+1:"

Oct 25 '23 17:10 github-actions[bot]

@osamahammad21 does this have any effect on distributed routing? I don't see the frInst being serialized anywhere.

No. I don't serialize frInst. Also I ran gcd_nangate45_distributed.tcl test and it works fine.

Nov 06 '23 22:11 osamahammad21

@maliberty Although I see that dbInst::getLocation is not the same as dbInst::getTransform().getOffset(). I didn't spend much time trying to understand why but just wanted to point it out.

Nov 06 '23 22:11 osamahammad21

@maliberty Although I see that dbInst::getLocation is not the same as dbInst::getTransform().getOffset(). I didn't spend much time trying to understand why but just wanted to point it out.

See the comment in db.h start "IMPORTANT - INSTANCE PLACEMENT". In short they will be different if the orientation is not N

Nov 06 '23 22:11 maliberty