can't build?

/Volumes/mrs5/net/llvm/llvm/tools/clang/lib/Analysis/../../include/clang/Analysis/PathSensitive/GRExprEngine.h: In function ‘unsigned int clang::RunGRSimpleVals(clang::CFG&, clang::Decl&, clang::ASTContext&, clang::Diagnostic&, clang::PathDiagnosticClient*, bool, bool)’:
/Volumes/mrs5/net/llvm/llvm/tools/clang/lib/Analysis/../../include/clang/Analysis/PathSensitive/GRExprEngine.h:89: error: ‘typedef class llvm::DenseMap<clang::ExplodedNode<clang::ValueState>*, clang::Expr*, llvm::DenseMapInfo<clang::ExplodedNode<clang::ValueState>*>, llvm::DenseMapInfo<clang::Expr*> > clang::GRExprEngine::UndefArgsTy’ is protected
GRSimpleVals.cpp:233: error: within this context
/Volumes/mrs5/net/llvm/llvm/tools/clang/lib/Analysis/../../include/clang/Analysis/PathSensitive/GRExprEngine.h:89: error: ‘typedef class llvm::DenseMap<clang::ExplodedNode<clang::ValueState>*, clang::Expr*, llvm::DenseMapInfo<clang::ExplodedNode<clang::ValueState>*>, llvm::DenseMapInfo<clang::Expr*> > clang::GRExprEngine::UndefArgsTy’ is protected
GRSimpleVals.cpp:234: error: within this context
/Volumes/mrs5/net/llvm/llvm/tools/clang/lib/Analysis/../../include/clang/Analysis/PathSensitive/GRExprEngine.h:89: error: ‘typedef class llvm::DenseMap<clang::ExplodedNode<clang::ValueState>*, clang::Expr*, llvm::DenseMapInfo<clang::ExplodedNode<clang::ValueState>*>, llvm::DenseMapInfo<clang::Expr*> > clang::GRExprEngine::UndefArgsTy’ is protected
GRSimpleVals.cpp:240: error: within this context
/Volumes/mrs5/net/llvm/llvm/tools/clang/lib/Analysis/../../include/clang/Analysis/PathSensitive/GRExprEngine.h:89: error: ‘typedef class llvm::DenseMap<clang::ExplodedNode<clang::ValueState>*, clang::Expr*, llvm::DenseMapInfo<clang::ExplodedNode<clang::ValueState>*>, llvm::DenseMapInfo<clang::Expr*> > clang::GRExprEngine::UndefArgsTy’ is protected
GRSimpleVals.cpp:241: error: within this context
/Volumes/mrs5/net/llvm/llvm/tools/clang/lib/Analysis/../../include/clang/Analysis/PathSensitive/GRExprEngine.h:88: error: ‘typedef class llvm::SmallPtrSet<clang::ExplodedNode<clang::ValueState>*, 2u> clang::GRExprEngine::UndefReceiversTy’ is protected
GRSimpleVals.cpp:247: error: within this context
/Volumes/mrs5/net/llvm/llvm/tools/clang/lib/Analysis/../../include/clang/Analysis/PathSensitive/GRExprEngine.h:88: error: ‘typedef class llvm::SmallPtrSet<clang::ExplodedNode<clang::ValueState>*, 2u> clang::GRExprEngine::UndefReceiversTy’ is protected
GRSimpleVals.cpp:248: error: within this context
make[2]: *** [/Volumes/mrs5/net/llvm/llvm-build/tools/clang/lib/Analysis/Debug/GRSimpleVals.o] Error 1
make[1]: *** [Analysis/.makeall] Error 2

?

Builds fine for me.

Are you sure the llvm tree is up-to-date?

snaroff

Thanks Mike. I've submitted a patch that hopefully fixes this. It builds fine with gcc-4.0. It looks like your version of gcc is complaining about the typedefs being private.

Ted Kremenek wrote:

Thanks Mike. I've submitted a patch that hopefully fixes this. It
builds fine with gcc-4.0. It looks like your version of gcc is
complaining about the typedefs being private.

I get a different build error, see below. Attached patch fixes it. Ok to
commit?

make: Entering directory `/home/edwin/llvm-svn/llvm/tools/clang/lib/CodeGen'
llvm[0]: Compiling CGBuiltin.cpp for Release build
CGBuiltin.cpp: In member function ‘llvm::Value*
clang::CodeGen::CodeGenFunction::EmitX86BuiltinExpr(unsigned int, const
clang::CallExpr*)’:
CGBuiltin.cpp:478: error: jump to case label
CGBuiltin.cpp:473: error: crosses initialization of ‘unsigned int i’
CGBuiltin.cpp:482: error: jump to case label
CGBuiltin.cpp:473: error: crosses initialization of ‘unsigned int i’
make: ***
[/home/edwin/llvm-svn/llvm/tools/clang/lib/CodeGen/Release/CGBuiltin.o]
Error 1
make: Leaving directory `/home/edwin/llvm-svn/llvm/tools/clang/lib/CodeGen'

Best regards,
--Edwin

buildfix.patch (882 Bytes)

I think this is an unrelated problem, and I think this was introduced after I fixed the problem Mike was reporting. Please commit your fix!

Thanks!

Ted Kremenek wrote:

I think this is an unrelated problem, and I think this was introduced
after I fixed the problem Mike was reporting. Please commit your fix!

Done, r49209.

Best regards,
--Edwin