Added a graph exclusion clear for the mux to prevent previous graph creations causing bugs.

This commit is contained in:
Hunter Nichols 2021-06-14 14:39:54 -07:00
parent 7df36a916b
commit 74b55ea83b
6 changed files with 21 additions and 8 deletions

View File

@ -246,7 +246,7 @@ class delay(simulation):
bl_and_port = self.bl_name.format(port) # bl_name contains a '{}' for the port
# Isolate the s_en and bitline paths
debug.info(1, "self.bl_name = {0}".format(self.bl_name))
debug.info(1, "self.graph.all_paths = {0}".format(self.graph.all_paths))
debug.info(2, "self.graph.all_paths = {0}".format(self.graph.all_paths))
sen_paths = [path for path in self.graph.all_paths if sen_and_port in path]
bl_paths = [path for path in self.graph.all_paths if bl_and_port in path]
debug.check(len(sen_paths)==1, 'Found {0} paths which contain the s_en net.'.format(len(sen_paths)))

View File

@ -534,9 +534,9 @@ class simulation():
self.sram.graph_exclude_bits(self.wordline_row, self.bitline_column)
port=0 #FIXME, port_data requires a port specification, assuming single port for now
if self.words_per_row > 1:
self.sram.graph_clear_column_mux(port)
self.sram.graph_exclude_column_mux(self.bitline_column, port)
debug.info(0, "self.bitline_column={}".format(self.bitline_column))
# Generate new graph every analysis as edges might change depending on test bit
self.graph = graph_util.timing_graph()
self.sram_instance_name = "X{}".format(self.sram.name)
@ -558,7 +558,6 @@ class simulation():
"""
net_found = False
for path in paths:
debug.info(0, "path={}".format(path))
aliases = self.sram.find_aliases(self.sram_instance_name, self.pins, path, internal_net, mod, exclusion_set)
if net_found and len(aliases) >= 1:
debug.error('Found multiple paths with {} net.'.format(internal_net), 1)

View File

@ -1110,3 +1110,9 @@ class bank(design.design):
Excludes all columns muxes unrelated to the target bit being simulated.
"""
self.port_data[port].graph_exclude_column_mux(column_include_num)
def graph_clear_column_mux(self, port):
"""
Clear mux exclusions to allow different bit tests.
"""
self.port_data[port].graph_clear_column_mux()

View File

@ -236,10 +236,6 @@ class column_mux_array(design.design):
Excludes all columns muxes unrelated to the target bit being simulated.
Each mux in mux_inst corresponds to respective column in bitcell array.
"""
#stop = 34
for i in range(len(self.mux_inst)):
if i != column_include_num:
self.graph_inst_exclude.add(self.mux_inst[i])
debug.info(0, "Excluded mux {}".format(i))
#if i == stop:
# break

View File

@ -863,3 +863,10 @@ class port_data(design.design):
"""
if self.column_mux_array:
self.column_mux_array.graph_exclude_columns(column_include_num)
def graph_clear_column_mux(self):
"""
Clear mux exclusions to allow different bit tests.
"""
if self.column_mux_array:
self.column_mux_array.init_graph_params()

View File

@ -780,3 +780,8 @@ class sram_base(design, verilog, lef):
"""
self.bank.graph_exclude_column_mux(column_include_num, port)
def graph_clear_column_mux(self, port):
"""
Clear mux exclusions to allow different bit tests.
"""
self.bank.graph_clear_column_mux(port)