Andy Lester wrote:
This patch

http://www.parrotvm.org/svn/parrot/revision?rev=34423

is not a long-term solution. The headerizer has to get run at will. Infinoid, can you tell me more about those ifdefs so I can make the headerizer happy?

Happy to. The functions in question (is_ins_save and _is_ins_save) are declared as static, but conditionally defined inside of an #ifdef. If the prototypes are not put in the same #ifdefs, the following warnings result:

compilers/imcc/optimizer.c:87: warning: '_is_ins_save' declared 'static' but never defined compilers/imcc/optimizer.c:153: warning: 'is_ins_save' declared 'static' but never defined

This has shown up a couple of times in RT, most recently as #53990.

Mark

Reply via email to