opencos-eda 0.3.2__py3-none-any.whl → 0.3.3__py3-none-any.whl

opencos/eda_config_defaults.yml

@@ -145,6 +145,7 @@ tools:
       --timescale=1ns/1ns
       --allow-dup-initial-drivers
       -Wno-missing-top
+    compile-waivers: []
 
   verilator:
     defines: { }
@@ -153,11 +154,11 @@ tools:
       - "%Fatal"
     log-must-strings:
       - "R e p o r t: Verilator" # These come at end of compile or simulation, since we don't suppress it.
+    # Note: -j $(nproc) is handled in verilator.py, but you can override it in compile-args
     compile-args: |
       --timing
       --assert
       --autoflush
-      -j 2
       -sv
     compile-coverage-args: |
       --coverage

opencos/tests/helpers.py

@@ -242,7 +242,9 @@ class Helpers:
 
         return rc
 
-    def is_in_log(self, *want_str, logfile=None, windows_path_support=False):
+    def is_in_log(
+            self, *want_str, logfile=None, windows_path_support: bool = False
+    ) -> bool:
         '''Check if want_str (joined) is in the logfile, or self.DEFAULT_LOG'''
         logfile = self._resolve_logfile(logfile)
         want_str0 = ' '.join(list(want_str))
@@ -254,7 +256,9 @@ class Helpers:
                     return True
         return False
 
-    def get_log_lines_with(self, *want_str, logfile=None, windows_path_support=False):
+    def get_log_lines_with(
+            self, *want_str, logfile=None, windows_path_support: bool = False
+    ) -> list:
         '''gets all log lines with any of want_str args are in the logfile, or self.DEFAULT_LOG'''
         logfile = self._resolve_logfile(logfile)
         ret_list = []
@@ -267,6 +271,21 @@ class Helpers:
                     ret_list.append(line)
         return ret_list
 
+    def get_log_lines_with_all(
+            self, *want_str, logfile=None, windows_path_support: bool = False
+    ) -> list:
+        '''gets all log lines with ALL of want_str args are in the logfile, or self.DEFAULT_LOG'''
+        logfile = self._resolve_logfile(logfile)
+        ret_list = []
+        with open(logfile, encoding='utf-8') as f:
+            for line in f.readlines():
+                if all(x in line for x in list(want_str)):
+                    ret_list.append(line)
+                elif windows_path_support and \
+                     all(x.replace('/', '\\') in line for x in list(want_str)):
+                    ret_list.append(line)
+        return ret_list
+
     def get_log_words_with(self, *want_str, logfile=None, windows_path_support=False):
         '''gets all log words with any of *want_str within a single word
         in the logfile or self.DEFAULT_LOG

opencos/tools/slang.py

@@ -128,6 +128,7 @@ class CommandElabSlang(CommandElab, ToolSlang):
         command_list += self.tool_config.get('compile-args', '--single-unit').split()
         command_list += self.args['slang-args'] # add user args.
         command_list += self._get_slang_json_args(command_exe=command_list[0])
+        command_list += self._get_slang_tool_config_waivers()
 
         # incdirs
         for value in self.incdirs:
@@ -214,6 +215,11 @@ class CommandElabSlang(CommandElab, ToolSlang):
 
         return command_list
 
+    def _get_slang_tool_config_waivers(self) -> list:
+        # Add compile waivers from config and command-line args
+        return [f'-Wno-{waiver}' for waiver in
+                self.tool_config.get('compile-waivers', []) + self.args['compile-waivers']]
+
 
 class CommandLintSlang(CommandElabSlang):
     '''CommandLintSlang is a command handler for: eda lint --tool=slang.'''

opencos/tools/verilator.py

@@ -5,10 +5,12 @@ Contains classes for ToolVerilator and VerilatorSim, VerilatorElab.
 
 # pylint: disable=R0801 # (calling functions with same arguments)
 
+import multiprocessing
 import os
 import shutil
 import subprocess
 
+
 from opencos import util
 from opencos.eda_base import Tool
 from opencos.commands import CommandSim
@@ -210,15 +212,9 @@ class VerilatorSim(CommandSim, ToolVerilator):
 
         verilate_command_list = self._get_start_verilator_command_list(lint_only=lint_only)
 
-        # Add compile args from our self.config (tools.verilator.compile-args str)
-        config_compile_args = self.tool_config.get(
-            'compile-args',
-            '--timing --assert --autoflush -sv').split()
-        verilate_command_list += config_compile_args
-
         verilate_command_list += self._get_verilator_tool_config_waivers()
 
-        verilate_command_list += self._verilator_args_deal_with_cflags()
+        verilate_command_list += self._verilator_args_defaults_cflags_nproc()
 
         verilate_command_list += self._get_verilator_waves_args(lint_only=lint_only)
 
@@ -425,45 +421,51 @@ class VerilatorSim(CommandSim, ToolVerilator):
         return ret
 
 
-    def _verilator_args_deal_with_cflags(self) -> list:
+    def _verilator_args_defaults_cflags_nproc(self) -> list:
         '''Returns list of args to be added to verilator (compile) step
 
-        Uses self.args['verilate-args'] and self.args['compile-args']
+        Uses self.args['verilate-args'], self.args['compile-args'], and self.tool_config
+
+        Sets -j <value> and -CFLAGS -O<value> if not present in --config-yml, --compile-args,
+        or --verilate-args. If present, chooses the first instance (does not present duplicates
+        to verilator call).
         '''
 
-        # We can only support one -CFLAGS followed by one -O[0-9] arg in self.args['verilate-args']:
-        verilate_cflags_args_dict = {}
-        verilate_args = [] # will be combined verilate_args + compile-args
-        prev_arg_is_cflags = False
+        # We can only support one -CFLAGS followed by one -O[0-9] arg in
+        # --verilate-args or --compile-args.
+
+        # Add compile args from our self.config (tools.verilator.compile-args str)
+        verilate_args = self.args['verilate-args'] + \
+            self.args['compile-args'] + \
+            self.tool_config.get(
+                'compile-args',
+                '--timing --assert --autoflush -sv').split()
+
         util.debug(f"{self.args['verilate-args']=}")
         util.debug(f"{self.args['compile-args']=}")
-        for arg in self.args['verilate-args'] + self.args['compile-args']:
-            # pick the first ones we see of these:
-            if arg == '-CFLAGS':
-                prev_arg_is_cflags = True
-                if arg not in verilate_cflags_args_dict:
-                    # We can only have 1
-                    verilate_cflags_args_dict[arg] = True
-                    verilate_args.append(arg)
-                else:
-                    util.debug(f'Previous saw -CFLAGS args {verilate_cflags_args_dict=},',
-                               f'skipping new {arg=}')
-
-            elif arg.startswith('-O') and len(arg) == 3:
-                if '-O' not in verilate_cflags_args_dict and prev_arg_is_cflags:
-                    # We can only have 1
-                    verilate_cflags_args_dict['-O'] = arg[-1]
-                    verilate_args.append(arg)
-                else:
-                    util.debug(f'Previous saw -CFLAGS args {verilate_cflags_args_dict=},',
-                               f'skipping new {arg=}')
-                prev_arg_is_cflags = False
-
-            else:
-                prev_arg_is_cflags = False
-                verilate_args.append(arg)
 
-        if '-CFLAGS' in verilate_args:
+        dash_j_arg_indices = []
+        cflags_dasho_args_indices = []
+        for i, arg in enumerate(list(verilate_args)):
+            # There can only be one of these: -j <value>, similarly can only be one of
+            # -CFLAGS -O<value>
+            if (i + 1) < len(verilate_args):
+                if arg == '-j':
+                    dash_j_arg_indices.extend([i, i + 1])
+                if arg == '-CFLAGS':
+                    next_arg = verilate_args[i + 1]
+                    if next_arg.startswith('-O') and len(next_arg) == 3:
+                        cflags_dasho_args_indices.extend([i, i + 1])
+
+        # For -j <value> we'll pick the first one, remove the rest.
+        # Same goes for -CFLAGS -O<value>
+        for index in dash_j_arg_indices[2:] + cflags_dasho_args_indices[2:]:
+            verilate_args[index] = ''
+
+        verilate_args = [x for x in verilate_args if x != ''] # strip empty str.
+
+        # Support for --optimize which will use -CFLAGS -O3, if -CFLAGS is not present at all.
+        if cflags_dasho_args_indices:
             # add whatever args were passed via 'compile-args' or 'verilate_args'. Note these will
             # take precedence over the --optimize arg.
             pass
@@ -472,8 +474,15 @@ class VerilatorSim(CommandSim, ToolVerilator):
             # (slower compile, better runtime)
             verilate_args += '-CFLAGS', '-O3'
         else:
+            # Default to -O1:
             verilate_args += '-CFLAGS', '-O1'
 
+        # If there was no -j setting, then use max(2, $(nproc) - 1)
+        if not dash_j_arg_indices:
+            nproc = max(2, multiprocessing.cpu_count() - 1)
+            verilate_args += '-j', f'{nproc}'
+
+
         return verilate_args
 
 

opencos/util.py

@@ -18,6 +18,7 @@ from enum import Enum
 from pathlib import Path
 from importlib import import_module
 from dotenv import load_dotenv
+from supports_color import supportsColor
 
 from opencos.utils import status_constants
 
@@ -29,7 +30,7 @@ dot_f_files_expanded = set() # pylint: disable=invalid-name
 env_files_loaded = set()     # pylint: disable=invalid-name
 
 args = {                    # pylint: disable=invalid-name
-    'color' : False,
+    'color' : bool(supportsColor.stdout),
     'quiet' : False,
     'verbose' : False,
     'debug' : False,
@@ -351,13 +352,14 @@ def get_argparser() -> argparse.ArgumentParser:
     bool_action_kwargs = get_argparse_bool_action_kwargs()
 
     parser.add_argument('--version', default=False, action='store_true')
-    parser.add_argument('--color', **bool_action_kwargs, default=True,
+    parser.add_argument('--color', **bool_action_kwargs, default=bool(supportsColor.stdout),
                         help='Use shell colors for info/warning/error messaging')
-    parser.add_argument('--quiet', **bool_action_kwargs, help='Do not display info messaging')
-    parser.add_argument('--verbose', **bool_action_kwargs,
+    parser.add_argument('--quiet', **bool_action_kwargs, default=args['quiet'],
+                        help='Do not display info messaging')
+    parser.add_argument('--verbose', **bool_action_kwargs, default=args['verbose'],
                         help='Display additional messaging level 2 or higher')
-    parser.add_argument('--fancy', **bool_action_kwargs)
-    parser.add_argument('--debug', **bool_action_kwargs,
+    parser.add_argument('--fancy', **bool_action_kwargs, default=args['fancy'])
+    parser.add_argument('--debug', **bool_action_kwargs, default=args['debug'],
                         help='Display additional debug messaging level 1 or higher')
     parser.add_argument('--debug-level', type=int, default=0,
                         help='Set debug level messaging (default: 0)')

{opencos_eda-0.3.2.dist-info → opencos_eda-0.3.3.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: opencos-eda
-Version: 0.3.2
+Version: 0.3.3
 Summary: A simple Python package for wrapping RTL simuliatons and synthesis
 Author-email: Simon Sabato <simon@cognichip.ai>, Drew Ranck <drew@cognichip.ai>
 Project-URL: Homepage, https://github.com/cognichip/opencos
@@ -17,4 +17,5 @@ Requires-Dist: toml>=0.10.2
 Requires-Dist: yamllint>=1.35.1
 Requires-Dist: PySerial>=3.5
 Requires-Dist: cocotb>=2.0
+Requires-Dist: supports_color>=0.2.0
 Dynamic: license-file

{opencos_eda-0.3.2.dist-info → opencos_eda-0.3.3.dist-info}/RECORD

@@ -5,7 +5,7 @@ opencos/deps_schema.py,sha256=VUdXuq43mKfM-U4x7DSA28-MH1Xqxre6V7Ttw2DeOqI,16762
 opencos/eda.py,sha256=91E-EsyZS-uRadApP-h2onW6rpvLBnrpJoT_9tRtsS8,23322
 opencos/eda_base.py,sha256=jf4t11UPj39swL41z-EJiGTOnHvRScaQc-SAjpZ5XI4,109651
 opencos/eda_config.py,sha256=z3yQOPGBX7-yKp6BdQYfJ9eOJf-Jctl-mwCDj3vj2BI,12712
-opencos/eda_config_defaults.yml,sha256=LF0yAncYeaPtZIoAfEeo2aiDXT4cjYa99soGks0WRzM,16063
+opencos/eda_config_defaults.yml,sha256=Bs9UyXckIpauK_tf27iRkig6hBvsuI-PL7HhgCQpA1k,16168
 opencos/eda_config_max_verilator_waivers.yml,sha256=lTAU4IOEbUWVlPzuer1YYhIyxpPINeA4EJqcRIT-Ymk,840
 opencos/eda_config_reduced.yml,sha256=cQ9jY4J7EvAbeHTiP6bvpDSVJAYiitjLZPSxxLKIEbk,1440
 opencos/eda_deps_bash_completion.bash,sha256=jMkQKY82HBgOnQeMdA1hMrXguRFtB52SMBxUemKovL4,1958
@@ -18,7 +18,7 @@ opencos/files.py,sha256=AQOnsrvoc0r76LiFrkoMbwOGdUO1FpBiFY_jyyI_ve8,1566
 opencos/names.py,sha256=Y2aJ5wgpbNIJ-_P5xUXnHMv_h-zMOX2Rt6iLuduqC1Q,1213
 opencos/peakrdl_cleanup.py,sha256=vHNGtalTrIVP335PhRjPt9RhoccgpK1HJAi-E4M8Kc8,736
 opencos/seed.py,sha256=IL9Yg-r9SLSRseMVWaEHmuw2_DNi_eyut11EafoNTsU,942
-opencos/util.py,sha256=ffaSoDDOsL6mx_fYgAyJA8fawsxZ5YHKqs67b8iW5rw,41921
+opencos/util.py,sha256=tgDOvkIGCPfuuJmP1nMsxsxaxmhlH9DOUsJ3SVjtXy4,42123
 opencos/commands/__init__.py,sha256=oOOQmn5_jHAMSOfA3swJJ7mdoyHsJA0lJwKPTudlTns,1125
 opencos/commands/build.py,sha256=mvJYxk5J15k0Cr8R7oIdIIdsEtWV3gE-LnPweVwtSDo,1487
 opencos/commands/deps_help.py,sha256=WDrU7H9sypzDAxe_CHqhW5B_scbQMzBEdf-v-Jcfd5Q,10682
@@ -47,7 +47,7 @@ opencos/hw/oc_cli.py,sha256=U1JGlshLZhtd0LgndZFBZVltAj_HemdhbjO_Zo8ZuVM,132252
 opencos/hw/pcie.py,sha256=VUJljaZJYgScAAx5yn7F6GoA8K9eTcw24otYZbkMpYs,3035
 opencos/tests/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 opencos/tests/custom_config.yml,sha256=TRoVM9ZFKPOA_8JmlpzaMhnGO1txmaD14N_8P1oqzew,257
-opencos/tests/helpers.py,sha256=wkEzYRt78B4b7Qni2GoWcncrhWjZ3nLhDMHtPYvvW8w,11725
+opencos/tests/helpers.py,sha256=0KWHUQNXeyQ40F2sdW_R2r0P8qZpV88ndTZmoix3NcU,12474
 opencos/tests/test_build.py,sha256=FQAxOpLVQShAHD_L5rqJctPeSAoqoOCNFI0RXflLuY0,387
 opencos/tests/test_deps_helpers.py,sha256=uQZxleh6aKO-mZQhagHh5xLIBbpQ8dav7-5D0eemq_g,8164
 opencos/tests/test_deps_schema.py,sha256=T3P9KjaMyKsk8b7snNVvNSsom2hIJcg6Z9apYiXoH9Y,941
@@ -74,11 +74,11 @@ opencos/tools/quartus.py,sha256=_TfmPSYpbhmDLw7Dur-rRP0iGwv9hhQ6E5G-XLiYPEM,3048
 opencos/tools/questa.py,sha256=nHImM0Wydcf4YHGibHmQAwmqKHmMxKZUqY-E-vz1o8M,9827
 opencos/tools/questa_fse.py,sha256=hytkeuGg4qImj7rStV1i2kxkz9B0KFheGtcadxmpYAo,2550
 opencos/tools/riviera.py,sha256=FAumXIt9u4JXwR5BHe-0APX3K0Lg3RT2lWQ-GNGL3vA,13016
-opencos/tools/slang.py,sha256=UpsFeExup2awARxxxADD9f923Hw6ubiSEq2tKtJbS1c,8305
+opencos/tools/slang.py,sha256=S_vODMT5Zl5vi9FMGHfahp5B0oMNyDIRJXtRAldVCwY,8625
 opencos/tools/slang_yosys.py,sha256=MKh13eAmLJDkynZiezyT8E2gI4CKnXipzgFCZppaMXo,10230
 opencos/tools/surelog.py,sha256=S2RAZJyjdISm_tRvAhXbla7_z_tJfotZih5f9Y3m7DQ,5648
 opencos/tools/tabbycad_yosys.py,sha256=2LePPgYXBVdsy7YcffPIWN-I0B7queLQ_f_pme2SCGw,7803
-opencos/tools/verilator.py,sha256=nVYF9F-VAHmT0uqmHpYZIvvZ2GjFIVW5zKpYBAWW8rI,20810
+opencos/tools/verilator.py,sha256=yh3DZXhGxt9RyJVtbn2RoqIiwxZW6_sTiXs8xGf3iNg,21065
 opencos/tools/vivado.py,sha256=GQdPd1mp02it_uyhMqpFfJgh0AKIMn-20BhHNEDp5HY,41398
 opencos/tools/yosys.py,sha256=t3Au8gdwTepIKCPCXHpRXEdtmORQK8xqNvF6baIa7DM,28260
 opencos/utils/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -88,10 +88,10 @@ opencos/utils/str_helpers.py,sha256=726ScK5-v7QkBi-zqESKZLsOl2_ya4vVJ5ZhxJqmBFo,
 opencos/utils/subprocess_helpers.py,sha256=xemAGPey6M0sWY_FElvr-Z0phCfdjaC-znP8FKihPaE,3535
 opencos/utils/vscode_helper.py,sha256=9nHyMUIL-gzfW-qLH06sgaCnVK-YTOtu6pusitNNhL8,1363
 opencos/utils/vsim_helper.py,sha256=1johPOGbjbMgnCDSTpgsQcSuAquiqq1Y2MBxS6WY6b4,1552
-opencos_eda-0.3.2.dist-info/licenses/LICENSE,sha256=HyVuytGSiAUQ6ErWBHTqt1iSGHhLmlC8fO7jTCuR8dU,16725
-opencos_eda-0.3.2.dist-info/licenses/LICENSE.spdx,sha256=8gn1610RMP6eFgT3Hm6q9VKXt0RvdTItL_oxMo72jII,189
-opencos_eda-0.3.2.dist-info/METADATA,sha256=0PgS7CqBIdwYkBXPQ_6PaCFCNFKEsRz5G_owk48zkJI,666
-opencos_eda-0.3.2.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-opencos_eda-0.3.2.dist-info/entry_points.txt,sha256=6n1T5NwVYDhN5l1h5zmyT197G4pE0SySDreB0QJzJR0,218
-opencos_eda-0.3.2.dist-info/top_level.txt,sha256=J4JDP-LpRyJqPNeh9bSjx6yrLz2Mk0h6un6YLmtqql4,8
-opencos_eda-0.3.2.dist-info/RECORD,,
+opencos_eda-0.3.3.dist-info/licenses/LICENSE,sha256=HyVuytGSiAUQ6ErWBHTqt1iSGHhLmlC8fO7jTCuR8dU,16725
+opencos_eda-0.3.3.dist-info/licenses/LICENSE.spdx,sha256=8gn1610RMP6eFgT3Hm6q9VKXt0RvdTItL_oxMo72jII,189
+opencos_eda-0.3.3.dist-info/METADATA,sha256=MGcmalurglc-9-KPz1YHp-KOVgLwqOZRRu3cOz6asI0,703
+opencos_eda-0.3.3.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+opencos_eda-0.3.3.dist-info/entry_points.txt,sha256=6n1T5NwVYDhN5l1h5zmyT197G4pE0SySDreB0QJzJR0,218
+opencos_eda-0.3.3.dist-info/top_level.txt,sha256=J4JDP-LpRyJqPNeh9bSjx6yrLz2Mk0h6un6YLmtqql4,8
+opencos_eda-0.3.3.dist-info/RECORD,,