From 97b8ca2ee421f2e10ad21b1bd96a652641ba62cf Mon Sep 17 00:00:00 2001 From: Gabe Black Date: Wed, 27 Oct 2021 01:25:44 -0700 Subject: [PATCH] sim: Remove some old transitional code in SEWorkload. This code was just to catch cases where the SEWorkload init_compatible function couldn't find a compatible SEWorkload subclass. Now that all of these classes are set up with this mechanism, there's no need to keep this code around. Change-Id: Ie847f5a90ccf98eb58c149a22a6881529344946d Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/52107 Reviewed-by: Giacomo Travaglini Maintainer: Giacomo Travaglini Tested-by: kokoro --- src/sim/Workload.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/sim/Workload.py b/src/sim/Workload.py index bd1a230071..52bda94ff2 100644 --- a/src/sim/Workload.py +++ b/src/sim/Workload.py @@ -102,10 +102,6 @@ class SEWorkload(Workload, metaclass=SEWorkloadMeta): if len(options) > 1: raise ValueError("More than one SE workload is compatible with %s") elif len(options) < 1: - # For now, fall back to the base class if there are no matches. - # After we've had a chance to implement everything, this default - # can be removed since this should always find exactly one match. - return SEWorkload(*args, **kwargs) raise ValueError("No SE workload is compatible with %s", path) return options[0](*args, **kwargs)