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
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Pkg-clamav-devel mailing list Pkg-clamav-devel@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-clamav-devel