[lldb-dev] -Werror and lldb

[Eric Christopher]

Cool deal. Appreciated. :)

-eric
On Sep 9, 2014 8:53 AM, "Colin Riley" <colin at codeplay.com> wrote:

>  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 listlldb-dev at cs.uiuc.eduhttp://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
>
>
>
> ------------------------------
>    <http://www.avast.com/>
>
> This email is free from viruses and malware because avast! Antivirus
> <http://www.avast.com/> protection is active.
>
>
> _______________________________________________
> lldb-dev mailing list
> lldb-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20140909/1df0d39b/attachment.html>