llvm dev - Dec 2008 - [LLVMdev] Build errors on trunk for about a week now.

Been trying to build the trunk to test some things for about a week
now using VS8 (VS2k5).  Tons of Warnings (like things first being
declared struct, being redefined class and so forth, those need to be
fixed, but are otherwise not harmful), and a *lot* of errors.  Being
trunk I figured just the normal trunk-type issues, but it has been
going on for a while now, so figured it would be good to state so it
can be fixed.  These errors are with a fresh CMake from a fresh SVN
checkout all into fresh directories, still happens.  Full build log is
attached to this email (too big for pastebin uncompressed, but
compressed down to a nice 15kb).

And I was going to put a copy of just the error lines in here, but
that came out to way too long, most of them seem to have been broken
from a couple distinct sources though.  Also, some of those warnings
are potentially harmful (not that LLVM ever built clean, most of them
are ignorable anyway).  If I get time during this break I may try to
clean up some of those irritating warnings that are simple (assuming
there are no errors stopping me, I do not think I will have time to
donate to those for another few weeks anyway...).
-------------- next part --------------
A non-text attachment was scrubbed...
Name: BuildLog_20081204.7z
Type: application/octet-stream
Size: 14999 bytes
Desc: not available
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20081204/530de38b/attachment.obj>

OvermindDL1 a écrit :
> Been trying to build the trunk to test some things for about a week
> now using VS8 (VS2k5).  Tons of Warnings (like things first being
> declared struct, being redefined class and so forth, those need to be
> fixed, but are otherwise not harmful), and a *lot* of errors.  Being
> trunk I figured just the normal trunk-type issues, but it has been
> going on for a while now, so figured it would be good to state so it
> can be fixed.  These errors are with a fresh CMake from a fresh SVN
> checkout all into fresh directories, still happens.  Full build log is
> attached to this email (too big for pastebin uncompressed, but
> compressed down to a nice 15kb).
>   
should be fixed with r60590 (work for me)

I hope this commit is considered as a trivial fix and that I didn't 
overstep my commit right.
I don't have a working test suit here, so I only checked it compile on 
VS2008.

On Fri, Dec 5, 2008 at 6:38 AM, Cédric Venet <cedric.venet at laposte.net>
wrote:
> should be fixed with r60590 (work for me)
That seems to have fixed a large amount of those errors (nicely simple
fix).  I went ahead and termserved into my dev box (I will not be able
to get to it for another day or so) and told svn to update, and cmake
to make into a new directory, and build it, but it is still getting
errors.  I will just post the error section and a couple surrounding
lines.

While building "LLVMSupport":
Annotation.cpp
f:\Program Files\Microsoft Visual Studio 8\VC\include\xtree(1174) :
error C3848: expression having type 'const
`anonymous-namespace'::StrCmp' would lose some const-volatile
qualifiers in order to call 'bool
`anonymous-namespace'::StrCmp::operator ()(const char *,const char *)'
        f:\Program Files\Microsoft Visual Studio
8\VC\include\xtree(1169) : while compiling class template member
function 'std::_Tree_nod<_Traits>::_Node
*std::_Tree<_Traits>::_Lbound(const char *const &) const'
        with
        [
            _Traits=std::_Tmap_traits<const char *,unsigned
int,`anonymous-namespace'::StrCmp,std::allocator<std::pair<const char
*const ,unsigned int>>,false>
        ]
        f:\Program Files\Microsoft Visual Studio 8\VC\include\map(82)
: see reference to class template instantiation
'std::_Tree<_Traits>'
being compiled
        with
        [
            _Traits=std::_Tmap_traits<const char *,unsigned
int,`anonymous-namespace'::StrCmp,std::allocator<std::pair<const char
*const ,unsigned int>>,false>
        ]
        ..\..\..\trunk\lib\Support\Annotation.cpp(67) : see reference
to class template instantiation 'std::map<_Kty,_Ty,_Pr>' being
compiled
        with
        [
            _Kty=const char *,
            _Ty=unsigned int,
            _Pr=`anonymous-namespace'::StrCmp
        ]
Allocator.cpp
APSInt.cpp

I cannot really look at it well since the termserv connection is
effectively less then modem speed, but I looked at Annotation.cpp from
the svn (also not fun, downloading at ~400bytes a second), but from a
quick look-see, I would bet that on line 34, changing:
    bool operator()(const char *a, const char *b) {
To:
    bool operator()(const char *a, const char *b) const {
Will fix that error.  Which makes sense, all functor adapters for the
STL things like map are supposed to be const qualified anyway.  Yes, I
have encountered that error before, yes that was the exact type of fix
that fixed it any time I have ever encountered it.

I do not know if there are any others errors yet.  I will try to look
again once the build finishes and I can transfer the build log to
local (thankfully not needing the screen updates now that I have the
build started).