[lldb-dev] -Werror and lldb

[Colin Riley]

Hey Eric - thanks for the heads up. I'll look into getting this sorted :)

Cheers,
Colin


On 09/09/2014 08:03, Eric Christopher wrote:
> So for fun a bit ago I decided to see what it would take to build lldb 
> -Werror free as a subproject of llvm (in llvm/tools alongside clang of 
> course) and fixed up a bunch of warnings/errors that happen.
>
> lldb-mi has the last set of warnings that I wasn't able to trivially 
> fix up as it would require rewriting a bit of it so I'll just send 
> mail to the authors and hope it happens :)
>
> Here's the current problem:
>
> /usr/local/google/home/echristo/sources/llvm/tools/lldb/tools/lldb-mi/MIUtilVariant.h:120:16: 
> error: 'CMIUtilVariant::CDataObject<CMIUtilString>::Copy' hides 
> overloaded virtual
>       function [-Werror,-Woverloaded-virtual]
>                 virtual void Copy( const CDataObject & vrOther );
>                              ^
> /usr/local/google/home/echristo/sources/llvm/tools/lldb/tools/lldb-mi/MIUtilVariant.h:266:22: 
> note: in instantiation of template class 
> 'CMIUtilVariant::CDataObject<CMIUtilString>'
>       requested here
>         m_pDataObject = new CDataObject< T >(  vArg );
>                             ^
> /usr/local/google/home/echristo/sources/llvm/tools/lldb/tools/lldb-mi/MIUtilMapIdToVariant.h:99:8: 
> note: in instantiation of function template specialization
>       'CMIUtilVariant::Set<CMIUtilString>' requested here
>                 data.Set< T >( vData );
>                      ^
> /usr/local/google/home/echristo/sources/llvm/tools/lldb/tools/lldb-mi/MICmnLLDBDebugSessionInfo.h:205:28: 
> note: in instantiation of function template specialization
>       'CMIUtilMapIdToVariant::Add<CMIUtilString>' requested here
>         if( !m_mapIdToSessionData.Add< T >( vKey, vData ) )
>                                   ^
> /usr/local/google/home/echristo/sources/llvm/tools/lldb/tools/lldb-mi/MICmdCmdEnviro.cpp:93:32: 
> note: in instantiation of function template specialization
> 'CMICmnLLDBDebugSessionInfo::SharedDataAdd<CMIUtilString>' requested here
>                 if( !m_rLLDBDebugSessionInfo.SharedDataAdd< 
> CMIUtilString >( rStrKeyWkDir, strWkDir ) )
>                                              ^
> /usr/local/google/home/echristo/sources/llvm/tools/lldb/tools/lldb-mi/MIUtilVariant.h:85:16: 
> note: hidden overloaded virtual function 
> 'CMIUtilVariant::CDataObjectBase::Copy' declared
>       here: type mismatch at 1st parameter ('const 
> CMIUtilVariant::CDataObjectBase &' vs 'const 
> CMIUtilVariant::CDataObject<CMIUtilString> &')
>                 virtual void Copy( const CDataObjectBase & vrOther );
>
>
> which requires a bit of rewriting to get past. It would be very nice 
> if this were to happen since it would allow lldb to take advantage of 
> the newer warnings etc while being able to be built as part of the 
> build bots a little easier as well.
>
> Thanks!
>
> -eric
>
>
> _______________________________________________
> lldb-dev mailing list
> lldb-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev

-- 
- Colin Riley
Games Technology Director

Codeplay Software Ltd
45 York Place, Edinburgh, EH1 3HP
Tel: 0131 466 0503
Fax: 0131 557 6600
Website: http://www.codeplay.com
Twitter: https://twitter.com/codeplaysoft



---
This email is free from viruses and malware because avast! Antivirus protection is active.
http://www.avast.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20140909/379c833c/attachment.html>