From 642e041f771dc09eb58c9991ee2860c46df8cb32 Mon Sep 17 00:00:00 2001 From: "Emil J. Tywoniak" Date: Fri, 8 Aug 2025 16:32:51 +0200 Subject: [PATCH] const2ast: fix for consistency with previous diagnostics behavior --- frontends/verilog/const2ast.cc | 13 ++----------- frontends/verilog/verilog_frontend.h | 1 - 2 files changed, 2 insertions(+), 12 deletions(-) diff --git a/frontends/verilog/const2ast.cc b/frontends/verilog/const2ast.cc index 5d906cb0f..9b197b356 100644 --- a/frontends/verilog/const2ast.cc +++ b/frontends/verilog/const2ast.cc @@ -44,21 +44,12 @@ YOSYS_NAMESPACE_BEGIN using namespace AST; using namespace VERILOG_FRONTEND; -std::string ConstParser::fmt_maybe_loc(std::string msg) { - std::string s; - - s += stringf("%s:%d:", loc.begin.filename->c_str(), loc.begin.line); - - s += msg; - return s; -} - void ConstParser::log_maybe_loc_error(std::string msg) { - log_error("%s", fmt_maybe_loc(msg).c_str()); + log_file_error(*loc.begin.filename, loc.begin.line, "%s", msg.c_str()); } void ConstParser::log_maybe_loc_warn(std::string msg) { - log_warning("%s", fmt_maybe_loc(msg).c_str()); + log_file_warning(*loc.begin.filename, loc.begin.line, "%s", msg.c_str()); } // divide an arbitrary length decimal number by two and return the rest diff --git a/frontends/verilog/verilog_frontend.h b/frontends/verilog/verilog_frontend.h index eabc0d46b..83c0e37a1 100644 --- a/frontends/verilog/verilog_frontend.h +++ b/frontends/verilog/verilog_frontend.h @@ -43,7 +43,6 @@ namespace VERILOG_FRONTEND struct ConstParser { AST::AstSrcLocType loc; private: - std::string fmt_maybe_loc(std::string msg); void log_maybe_loc_error(std::string msg); void log_maybe_loc_warn(std::string msg); // divide an arbitrary length decimal number by two and return the rest