IRGeneratorForStatements does not check for builtin when mangling user identifiers

Author: clonker

libsolidity/codegen/ir/IRGeneratorForStatements.cpp

@@ -84,10 +84,7 @@ struct CopyTranslate: public yul::ASTCopier
 		// from the Yul dialect we are compiling to. So we are assuming here that the builtin
 		// functions are identical. This should not be a problem for now since everything
 		// is EVM anyway.
-		if (m_dialect.findBuiltin(_name.str()))
-			return _name;
-		else
-			return yul::YulName{"usr$" + _name.str()};
+		return yul::YulName{"usr$" + _name.str()};
 	}
 
 	yul::Identifier translate(yul::Identifier const& _identifier) override

libsolidity/experimental/codegen/IRGeneratorForStatements.cpp

@@ -72,10 +72,7 @@ struct CopyTranslate: public yul::ASTCopier
 
 	yul::YulName translateIdentifier(yul::YulName _name) override
 	{
-		if (m_dialect.findBuiltin(_name.str()))
-			return _name;
-		else
-			return yul::YulName{"usr$" + _name.str()};
+		return yul::YulName{"usr$" + _name.str()};
 	}
 
 	yul::Identifier translate(yul::Identifier const& _identifier) override