Cool deal. Appreciated. :)
-eric
On Sep 9, 2014 8:53 AM, "Colin Riley" <co...@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-...@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@cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
>
>
_______________________________________________
lldb-dev mailing list
lldb-dev@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
