luke
/
verilator
mirror of https://github.com/verilator/verilator.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Improve adding model error message

This commit is contained in:
Wilson Snyder 2025-09-20 11:02:42 -04:00
parent bfc438c455
commit 19ca140165
2 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
include/verilated.cpp
View File

@ -2849,11 +2849,13 @@ void VerilatedContext::addModel(const VerilatedModel* modelp) {
// We look for time passing, as opposed to post-eval(), as embedded // We look for time passing, as opposed to post-eval(), as embedded
// models might get added inside initial blocks. // models might get added inside initial blocks.
if (VL_UNLIKELY(time())) if (VL_UNLIKELY(time())) {
VL_FATAL_MT( const std::string msg
"", 0, "", = "Adding model '"s + modelp->hierName()
"Adding model when time is non-zero. ... Suggest check time(), or for restarting" + "' when time is non-zero. ... Suggest check time(), or for restarting"
" model use a new VerilatedContext"); " model use a new VerilatedContext";
VL_FATAL_MT("", 0, "", msg.c_str());
}
threadPoolp(); // Ensure thread pool is created, so m_threads cannot change any more threadPoolp(); // Ensure thread pool is created, so m_threads cannot change any more
m_threadsInModels += modelp->threads(); m_threadsInModels += modelp->threads();

2
test_regress/t/t_wrapper_reuse_context_bad.out
View File

@ -1,2 +1,2 @@
%Error: Adding model when time is non-zero. ... Suggest check time(), or for restarting model use a new VerilatedContext %Error: Adding model 'TOP' when time is non-zero. ... Suggest check time(), or for restarting model use a new VerilatedContext
Aborting... Aborting...