Remove unnecessary pruning of double-quoting

In the past I was calling the ILANG_BACKEND::dump_const() to dump
values to an output stream. When these values were strings, the
function used to add quotes around them. The firrtl compiler, in turn,
escaped these quotes and the result was double-quoted strings which
were hard to read.

However I'm now calling design_entity->get_src_attribute() directly
and there is no additional quote being put around it, so we can
safely remove the unnecessary call to str.erase() here.

diff --git a/backends/firrtl/firrtl.cc b/backends/firrtl/firrtl.cc
index 0cac437aca13f79c08e26751fcc627a8cb6ea32d..3ef88666440192c86fe8d74693c4691a8d97cf11 100644 (file)
--- a/backends/firrtl/firrtl.cc
+++ b/backends/firrtl/firrtl.cc
@@ -47,11 +47,6 @@ std::string getFileinfo(const RTLIL::AttrObject *design_entity)
 {
        std::string src(design_entity->get_src_attribute());
        std::string fileinfo_str = src.empty() ? "" : "@[" + src + "]";
-
-       // Remove quotes from src attribute as firrtl automatically escapes and
-       // double-quotes them.
-       fileinfo_str.erase(std::remove(fileinfo_str.begin(), fileinfo_str.end(), '\"'), fileinfo_str.end());
-
        return fileinfo_str;
 }