diff options
| author | whitequark <whitequark@whitequark.org> | 2020-05-08 05:30:32 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-05-08 05:30:32 +0000 |
| commit | ebfdf61eb9703a16f949470f98509fdb405adf8f (patch) | |
| tree | c5d9c3b6dd337630c50f75fb98a87f6897ef6a9c /backends | |
| parent | 5b779f7f4ef0bf2c4ad3a412da24fad30b078626 (diff) | |
| parent | d86fc791f9016fa6dafa7af1758ef301136e9eec (diff) | |
| download | yosys-ebfdf61eb9703a16f949470f98509fdb405adf8f.tar.gz yosys-ebfdf61eb9703a16f949470f98509fdb405adf8f.tar.bz2 yosys-ebfdf61eb9703a16f949470f98509fdb405adf8f.zip | |
Merge pull request #2022 from Xiretza/fallthroughs
Avoid switch fall-through warnings
Diffstat (limited to 'backends')
| -rw-r--r-- | backends/cxxrtl/cxxrtl.cc | 12 | ||||
| -rw-r--r-- | backends/firrtl/firrtl.cc | 4 |
2 files changed, 11 insertions, 5 deletions
diff --git a/backends/cxxrtl/cxxrtl.cc b/backends/cxxrtl/cxxrtl.cc index e7711962f..f3ed3f623 100644 --- a/backends/cxxrtl/cxxrtl.cc +++ b/backends/cxxrtl/cxxrtl.cc @@ -1943,13 +1943,13 @@ struct CxxrtlWorker { case RTLIL::STa: break; + case RTLIL::STg: + log_cmd_error("Global clock is not supported.\n"); + // Handling of init-type sync rules is delegated to the `proc_init` pass, so we can use the wire // attribute regardless of input. case RTLIL::STi: log_assert(false); - - case RTLIL::STg: - log_cmd_error("Global clock is not supported.\n"); } } @@ -2346,16 +2346,22 @@ struct CxxrtlBackend : public Backend { case 6: worker.max_opt_level = true; worker.run_proc_flatten = true; + YS_FALLTHROUGH case 5: worker.run_opt_clean_purge = true; + YS_FALLTHROUGH case 4: worker.localize_public = true; + YS_FALLTHROUGH case 3: worker.elide_public = true; + YS_FALLTHROUGH case 2: worker.localize_internal = true; + YS_FALLTHROUGH case 1: worker.elide_internal = true; + YS_FALLTHROUGH case 0: break; default: diff --git a/backends/firrtl/firrtl.cc b/backends/firrtl/firrtl.cc index 40d05a036..f6dae1d8c 100644 --- a/backends/firrtl/firrtl.cc +++ b/backends/firrtl/firrtl.cc @@ -343,7 +343,7 @@ struct FirrtlWorker switch (dir) { case FD_INOUT: log_warning("Instance port connection %s.%s is INOUT; treating as OUT\n", cell_type.c_str(), log_signal(it->second)); - /* FALLTHRU */ + YS_FALLTHROUGH case FD_OUT: sourceExpr = firstName; sinkExpr = secondExpr; @@ -351,7 +351,7 @@ struct FirrtlWorker break; case FD_NODIRECTION: log_warning("Instance port connection %s.%s is NODIRECTION; treating as IN\n", cell_type.c_str(), log_signal(it->second)); - /* FALLTHRU */ + YS_FALLTHROUGH case FD_IN: sourceExpr = secondExpr; sinkExpr = firstName; |