From a0681e580f24d7d1b0a7d3c280a7337e14712755 Mon Sep 17 00:00:00 2001 From: Cary R Date: Tue, 30 Mar 2010 10:51:18 -0700 Subject: [PATCH] The SDF timing spec list can have zero or more elements. The SDF file format supports zero or more timing specs. Previously the parser was one or more. Things like tie high/low cells do not have any delay information so zero or more is needed and matches the standard. (cherry picked from commit 68b1273d2d355497a8e5f7568e97444be8d64d5f) --- vpi/sdf_parse.y | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/vpi/sdf_parse.y b/vpi/sdf_parse.y index cf2796ec5..b638e0278 100644 --- a/vpi/sdf_parse.y +++ b/vpi/sdf_parse.y @@ -188,7 +188,7 @@ cell_list cell : '(' K_CELL celltype cell_instance { sdf_select_instance($3, $4); /* find the instance in the design */} - timing_spec_list + timing_spec_list_opt ')' { free($3); if ($4) free($4); @@ -216,9 +216,9 @@ cell_instance $$ = strdup(""); } ; -timing_spec_list - : timing_spec_list timing_spec - | timing_spec +timing_spec_list_opt + : /* Empty */ + | timing_spec_list_opt timing_spec ; timing_spec