[Pkg-clamav-devel] build clamav with system LLVM

[Scott Kitterman]

On Friday, May 02, 2014 19:46:50 Andreas Cadhalpun wrote:
> Hi,
> 
> in the branch aca_llvm I have created a patch to build clamav with the
> system LLVM.
> I think that would be good, because of security and other bugfixes in
> LLVM (the internal version is based on 2.8, while testing has 3.3 as
> default and 3.4 available).
> As a bonus, this drastically reduces build time.
> 
> To fix FTBFS with new versions, mostly header names had to be changed,
> but two functions were removed, that I couldn't find a replacement for:
>   * Attribute::constructStackAlignmentFromInt used in
> libclamav/c++/bytecode2llvm.cpp
>   * JITExceptionHandling (defined as DwarfExceptionHandling) used in
> libclamav/c++/bytecode2llvm.cpp
> 
> I don't think this is a major problem, but
> constructStackAlignmentFromInt was added to fix a bug [1]. Hopefully
> this is fixed now in LLVM directly.
> 
> What do you think about this?

I think using the system LLVM is a great step forward.  Before we switch, 
however, I would send the change upstream for review.  You can ask them about 
the constructStackAlignmentFromInt issue at the same time.

This would also solve the array-bounds and pointer-cast-size-mismatch issues 
identified in http://qa.debian.org/bls/packages/c/clamav.html

I suspect they would appreciate the patch since using the system LLVM (or at 
least updating to a newer one) is definitely in their plans.

Scott K
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.alioth.debian.org/pipermail/pkg-clamav-devel/attachments/20140502/f22a7d6c/attachment.sig>