Skip to content

Commit a57ddfc

Browse files
committed
Move convert call above body_provided check
The previous version would have not called convert_single_method on excluded methods, meaning they would have missed some meta-information, e.g. parameter identifiers.
1 parent 0714def commit a57ddfc

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/goto-programs/lazy_goto_functions_map.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -173,6 +173,9 @@ class lazy_goto_functions_mapt final
173173
const key_type &name,
174174
symbol_table_baset &function_symbol_table) const
175175
{
176+
// Fill in symbol table entry body if not already done
177+
language_files.convert_lazy_method(name, function_symbol_table);
178+
176179
underlying_mapt::iterator it=goto_functions.find(name);
177180
if(it!=goto_functions.end())
178181
return *it;
@@ -190,9 +193,6 @@ class lazy_goto_functions_mapt final
190193
// Second chance: see if language_filest can provide a body:
191194
if(!body_provided)
192195
{
193-
// Fill in symbol table entry body if not already done
194-
language_files.convert_lazy_method(name, function_symbol_table);
195-
196196
// Create goto_functiont
197197
goto_convert_functionst convert_functions(
198198
function_symbol_table, message_handler);

0 commit comments

Comments
 (0)