Hi--

Also, I notice this in libclamav/c++/bytecode2llvm.cpp around line 1994:

    SMDiagnostic diag(ctx->file, ctx->line ? ctx->line : -1,
         ctx->col ? ctx->col-1 : -1,
         "", std::string(lines->linev[ctx->line-1], lines->linev[ctx->line]-1));

struct cli_bc_ctx from clamav-0.96/bytecode_priv.h defines line and col to be 
unsigned:

    unsigned line;
    unsigned col;

If you really want to use -1 as a flag, please change these to signed ints.  
Compiler warnings were:

/bin/sh ./libtool --silent --tag=CXX   --mode=compile c++ -DHAVE_CONFIG_H -I.  
-I./../.. -I./.. -I./../../ -I./llvm/include -I./llvm/include 
-D__STDC_LIMIT_MACROS -D__STDC_CONSTANT_MACROS -D_DEBUG -D_GNU_SOURCE 
-I/usr/local/include   -Woverloaded-virtual -pedantic -Wno-long-long -Wall -W 
-Wno-unused-parameter -Wwrite-strings -O2 -fno-strict-aliasing -pipe 
-march=pentium -march=i486 -c -o libclamavcxx_la-bytecode2llvm.lo `test -f 
'bytecode2llvm.cpp' || echo './'`bytecode2llvm.cpp
bytecode2llvm.cpp: In function `void cli_bytecode_debug_printsrc(const 
cli_bc_ctx*)':
bytecode2llvm.cpp:1994: warning: converting of negative value 
`-0x00000000000000001' to `unsigned int'
bytecode2llvm.cpp:1995: warning: converting of negative value 
`-0x00000000000000001' to `unsigned int'

Regards,
-- 
-Chuck

PS: I also don't quite understand why the duplicate arch setting of 
"-march=pentium -march=i486" is showing up.  My compiler env is getting only:

  CPUTYPE?=i586

...from /etc/make.conf (which turns into CFLAGS for ./configure's purposes), 
since I am never going to run this on something which isn't a Pentium or later. 
 If something thinks it knows the system architecture better and/or insists 
upon compiling for 486, that's fine, but in such a case the -march=pentium 
shouldn't be emitted, even if the later -march=i486 should take priority.

_______________________________________________
Help us build a comprehensive ClamAV guide: visit http://wiki.clamav.net
http://www.clamav.net/support/ml

Reply via email to