The-OpenROAD-Project / OpenSTA

OpenSTA engine
GNU General Public License v3.0
389 stars 170 forks source link

unset_output_delay asks for unknown argument #52

Closed quentinw91 closed 3 years ago

quentinw91 commented 3 years ago

Hi, I wrote the following constraint.sdc for gcd example (nangate45):

create_clock [get_ports clk]  -name core_clock  -period 2

#set output delay for all
set_output_delay -clock core_clock -max 1 [all_outputs]
set_output_delay -clock core_clock -min 0 [all_outputs]

#except for one...
unset_output_delay [get_ports resp_val]
set_output_delay -clock core_clock -max 0.5 [get_ports resp_val]
set_output_delay -clock core_clock -min 0   [get_ports resp_val]

When running opensta, I receive the following error : ... Error: unset_output_delay requires two positional arguments. number instances in verilog is 272 ...

I also tried (even though it is marked as optional in the manual)

unset_output_delay -clock core_clock [get_ports resp_val]

but same response. What am i missing ? Thanks, Quentin

jjcherry56 commented 3 years ago

You aren't missing anything; it is a bug. Fixed in 17eced5f unset_input_delay/unset_output_delay no delay arg

jjcherry56 commented 3 years ago

note that you can also use [delete_from_list [all_ports] [get_ports resp_val]]

quentinw91 commented 3 years ago

ok thank you !