C front-end: floating-point built-ins are in the symbol table

It should never be the case that no symbol for `__CPROVER_isgreaterf`,
`__CPROVER_isgreaterd` and similar floating-point built-ins is present
for these are declared in cprover_builtin_headers.h. This commit removes
dead code and adds an additional consistency check.

Author: tautschnig

src/goto-programs/builtin_functions.cpp

@@ -1370,23 +1370,17 @@ void goto_convertt::do_function_call_symbol(
     // append d or f for double/float
     name+=use_double?'d':'f';
 
+    DATA_INVARIANT(
+      ns.lookup(name).type == f_type,
+      "builtin declaration should match constructed type");
+
     symbol_exprt new_function=function;
     new_function.set_identifier(name);
     new_function.type()=f_type;
 
     code_function_callt function_call(lhs, new_function, new_arguments);
     function_call.add_source_location()=function.source_location();
 
-    if(!symbol_table.has_symbol(name))
-    {
-      symbolt new_symbol;
-      new_symbol.base_name=name;
-      new_symbol.name=name;
-      new_symbol.type=f_type;
-      new_symbol.location=function.source_location();
-      symbol_table.add(new_symbol);
-    }
-
     copy(function_call, FUNCTION_CALL, dest);
   }
   else if(