Hi guys,<br><br>I've gone ahead and committed 221315 which disables lldb-mi if Werror is specified in the makefile build as it doesn't build and it's been a couple of months.<div><br></div><div>Thanks.</div><div><br></div><div>-eric</div><br><div class="gmail_quote">On Tue Sep 09 2014 at 10:24:04 AM Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Cool deal. Appreciated. :)</p>
<p dir="ltr">-eric</p>
<div class="gmail_quote">On Sep 9, 2014 8:53 AM, "Colin Riley" <<a href="mailto:colin@codeplay.com" target="_blank">colin@codeplay.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
Hey Eric - thanks for the heads up. I'll look into getting this
sorted :)<br>
<br>
Cheers,<br>
Colin<br>
<br>
<br>
<div>On 09/09/2014 08:03, Eric Christopher
wrote:<br>
</div>
<blockquote type="cite">
<div dir="ltr">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.<br>
<br>
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 :)<br>
<br>
Here's the current problem:<br>
<br>
/usr/local/google/home/echristo/sources/llvm/tools/lldb/tools/lldb-mi/MIUtilVariant.h:120:16:
error: 'CMIUtilVariant::CDataObject<CMIUtilString>::Copy'
hides overloaded virtual<br>
function [-Werror,-Woverloaded-virtual]<br>
virtual void Copy( const CDataObject &
vrOther );<br>
^<br>
/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>'<br>
requested here<br>
m_pDataObject = new CDataObject< T >( vArg );<br>
^<br>
/usr/local/google/home/echristo/sources/llvm/tools/lldb/tools/lldb-mi/MIUtilMapIdToVariant.h:99:8:
note: in instantiation of function template specialization<br>
'CMIUtilVariant::Set<CMIUtilString>' requested here<br>
data.Set< T >( vData );<br>
^<br>
/usr/local/google/home/echristo/sources/llvm/tools/lldb/tools/lldb-mi/MICmnLLDBDebugSessionInfo.h:205:28:
note: in instantiation of function template specialization<br>
'CMIUtilMapIdToVariant::Add<CMIUtilString>'
requested here<br>
if( !m_mapIdToSessionData.Add< T >( vKey, vData )
)<br>
^<br>
/usr/local/google/home/echristo/sources/llvm/tools/lldb/tools/lldb-mi/MICmdCmdEnviro.cpp:93:32:
note: in instantiation of function template specialization<br>
'CMICmnLLDBDebugSessionInfo::SharedDataAdd<CMIUtilString>'
requested here<br>
if( !m_rLLDBDebugSessionInfo.SharedDataAdd<
CMIUtilString >( rStrKeyWkDir, strWkDir ) )<br>
^<br>
/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<br>
here: type mismatch at 1st parameter ('const
CMIUtilVariant::CDataObjectBase &' vs 'const
CMIUtilVariant::CDataObject<CMIUtilString> &')<br>
virtual void Copy( const CDataObjectBase &
vrOther );<br>
<br>
<br>
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.<br>
<div><br>
</div>
<div>Thanks!</div>
<div><br>
</div>
<div>-eric</div>
</div>
<br>
<fieldset></fieldset>
<br>
<pre>_______________________________________________
lldb-dev mailing list
<a href="mailto:lldb-dev@cs.uiuc.edu" target="_blank">lldb-dev@cs.uiuc.edu</a>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a>
</pre>
</blockquote>
<br>
<pre cols="72">--
- Colin Riley
Games Technology Director
Codeplay Software Ltd
45 York Place, Edinburgh, EH1 3HP
Tel: 0131 466 0503
Fax: 0131 557 6600
Website: <a href="http://www.codeplay.com" target="_blank">http://www.codeplay.com</a>
Twitter: <a href="https://twitter.com/codeplaysoft" target="_blank">https://twitter.com/codeplaysoft</a>
</pre>
<br><br>
<hr style="border:none;color:#909090;background-color:#b0b0b0;min-height:1px;width:99%">
<table style="border-collapse:collapse;border:none">
<tr>
<td style="border:none;padding:0px 15px 0px 8px">
<a href="http://www.avast.com/" target="_blank">
<img border="0" src="http://static.avast.com/emails/avast-mail-stamp.png">
</a>
</td>
<td>
<p style="color:#3d4d5a;font-family:"Calibri","Verdana","Arial","Helvetica";font-size:12pt">
This email is free from viruses and malware because <a href="http://www.avast.com/" target="_blank">avast! Antivirus</a> protection is active.
</p>
</td>
</tr>
</table>
<br>
</div>
<br>_______________________________________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@cs.uiuc.edu" target="_blank">lldb-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a><br>
<br></blockquote></div>
</blockquote></div>