<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<meta name="Generator" content="Microsoft Word 12 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:Wingdings;
panose-1:5 0 0 0 0 0 0 0 0 0;}
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
{font-family:Tahoma;
panose-1:2 11 6 4 3 5 4 4 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:12.0pt;
font-family:"Times New Roman","serif";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
p.MsoListParagraph, li.MsoListParagraph, div.MsoListParagraph
{mso-style-priority:34;
margin-top:0in;
margin-right:0in;
margin-bottom:0in;
margin-left:.5in;
margin-bottom:.0001pt;
font-size:12.0pt;
font-family:"Times New Roman","serif";}
span.EmailStyle17
{mso-style-type:personal-reply;
font-family:"Calibri","sans-serif";
color:#1F497D;}
.MsoChpDefault
{mso-style-type:export-only;}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
/* List Definitions */
@list l0
{mso-list-id:1006899851;
mso-list-type:hybrid;
mso-list-template-ids:-2091454652 699977576 67698691 67698693 67698689 67698691 67698693 67698689 67698691 67698693;}
@list l0:level1
{mso-level-start-at:0;
mso-level-number-format:bullet;
mso-level-text:\F0D8;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:Wingdings;
mso-fareast-font-family:Calibri;
mso-bidi-font-family:"Times New Roman";}
@list l1
{mso-list-id:1012491048;
mso-list-type:hybrid;
mso-list-template-ids:582113076 1552341626 67698691 67698693 67698689 67698691 67698693 67698689 67698691 67698693;}
@list l1:level1
{mso-level-start-at:0;
mso-level-number-format:bullet;
mso-level-text:-;
mso-level-tab-stop:none;
mso-level-number-position:left;
text-indent:-.25in;
font-family:"Calibri","sans-serif";
mso-fareast-font-family:Calibri;}
ol
{margin-bottom:0in;}
ul
{margin-bottom:0in;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoListParagraph" style="text-indent:-.25in;mso-list:l0 level1 lfo1"><![if !supportLists]><span style="font-family:Wingdings"><span style="mso-list:Ignore">Ø<span style="font:7.0pt "Times New Roman"">
</span></span></span><![endif]>-Wno-sign-compare -Wno-four-char-constants -Wno-extended-offsetof -Wno-unused-function<o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Do you need to disable these warnings when building with clang 3.4? The short list of remaining warnings that you quoted is more consistent with the clang
3.4 buildbot runs. Example:<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">
</span><a href="http://lab.llvm.org:8011/builders/lldb-x86_64-debian-clang/builds/4094/steps/compile/logs/warnings%20%286%29">http://lab.llvm.org:8011/builders/lldb-x86_64-debian-clang/builds/4094/steps/compile/logs/warnings%20%286%29</a><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">FYI, Matt has done a great job of fixing a lot of warnings to date. Example:<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">
</span><a href="http://llvm.org/viewvc/llvm-project?rev=183140&view=rev">http://llvm.org/viewvc/llvm-project?rev=183140&view=rev</a><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoListParagraph" style="text-indent:-.25in;mso-list:l1 level1 lfo2"><![if !supportLists]><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><span style="mso-list:Ignore">-<span style="font:7.0pt "Times New Roman"">
</span></span></span><![endif]><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">Ashok<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> lldb-dev-bounces@cs.uiuc.edu [mailto:lldb-dev-bounces@cs.uiuc.edu]
<b>On Behalf Of </b>Michael Sartain<br>
<b>Sent:</b> Wednesday, June 12, 2013 3:12 PM<br>
<b>To:</b> lldb-dev@cs.uiuc.edu<br>
<b>Subject:</b> [lldb-dev] lldb build warnings<o:p></o:p></span></p>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<p class="MsoNormal">How do folks handle warnings when building debug lldb?<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">I'm building on Linux with Clang 3.2 and had to add these options to get the number down to a reasonable count:<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">-Wno-sign-compare -Wno-four-char-constants -Wno-extended-offsetof -Wno-unused-function<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Still seeing several others though (down below). Having all these warnings can make it difficult to track down new ones though.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> -Mike<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<div>
<p class="MsoNormal">mikesart@mikesart-rad:~/data/src/llvm/build$ lldb_build<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">ninja: Entering directory `/home/mikesart/data/src/llvm/build'<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">[88/2270] cd /home/mikesart/data/src/llvm/build/tools/clang/lib/Basic && /usr/local/bin/cmak...ang/lib/Basic/SVNVersion.inc -P /home/mikesart/data/src/llvm/llvm/cmake/modules/GetSVN.cmak<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">-- Found Subversion: /usr/bin/svn (found version "1.7.9") <o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">[140/2270] Building lldb python wrapper<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">SWIG needs to be re-run.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">[633/2270] Building CXX object lib/Target/PowerPC/InstPrinter/CMakeFiles/LLVMPowerPCAsmPrinter.dir/PPCInstPrinter.cpp.o<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">/home/mikesart/data/src/llvm/llvm/lib/Target/PowerPC/InstPrinter/PPCInstPrinter.cpp:92:13: warning: enumeration value 'PRED_BAD' not handled in switch [-Wswitch]<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> switch ((PPC::Predicate)Code) {<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> ^<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">1 warning generated.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">[722/2270] Building CXX object lib/Target/PowerPC/MCTargetDesc/CMakeFiles/LLVMPowerPCDesc.dir/PPCPredicates.cpp.o<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">/home/mikesart/data/src/llvm/llvm/lib/Target/PowerPC/MCTargetDesc/PPCPredicates.cpp:20:11: warning: enumeration value 'PRED_BAD' not handled in switch [-Wswitch]<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> switch (Opcode) {<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> ^<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">/home/mikesart/data/src/llvm/llvm/lib/Target/PowerPC/MCTargetDesc/PPCPredicates.cpp:34:11: warning: enumeration value 'PRED_BAD' not handled in switch [-Wswitch]<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> switch (Opcode) {<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> ^<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">2 warnings generated.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">[1936/2270] Building CXX object tools/lldb/source/Core/CMakeFiles/lldbCore.dir/DataExtractor.cpp.o<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">/home/mikesart/data/src/llvm/llvm/tools/lldb/source/Core/DataExtractor.cpp:1328:22: warning: hexadecimal floating constants are a C99 feature [-Wc99-extensions]<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> return u.f * 0x1.0p125f;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> ^<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">/home/mikesart/data/src/llvm/llvm/tools/lldb/source/Core/DataExtractor.cpp:1333:18: warning: hexadecimal floating constants are a C99 feature [-Wc99-extensions]<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> return u.f * 0x1.0p-112f;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> ^<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">2 warnings generated.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">[2264/2270] Building CXX object tools/lldb/source/CMakeFiles/liblldb.dir/__/scripts/LLDBWrapPython.cpp.o<o:p></o:p></p>
</div>
</div>
<div>
<div>
<p class="MsoNormal">tools/lldb/scripts/LLDBWrapPython.cpp:2380:23: warning: explicitly assigning a variable of type 'int' to itself [-Wself-assign]<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> res = SWIG_AddCast(res);<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> ~~~ ^ ~~~<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">tools/lldb/scripts/LLDBWrapPython.cpp:2383:23: warning: explicitly assigning a variable of type 'int' to itself [-Wself-assign]<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> res = SWIG_AddCast(res); <o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> ~~~ ^ ~~~<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">tools/lldb/scripts/LLDBWrapPython.cpp:2905:9: warning: variable 'res' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized]<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> if (PyType_Ready(tp) < 0)<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> ^~~~~~~~~~~~~~~~~~~~<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">tools/lldb/scripts/LLDBWrapPython.cpp:2929:10: note: uninitialized use occurs here<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> return res;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> ^~~<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">tools/lldb/scripts/LLDBWrapPython.cpp:2905:5: note: remove the 'if' if its condition is always false<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> if (PyType_Ready(tp) < 0)<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> ^~~~~~~~~~~~~~~~~~~~~~~~~<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">tools/lldb/scripts/LLDBWrapPython.cpp:2886:10: note: initialize the variable 'res' to silence this warning<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> int res;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> ^<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> = 0<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">tools/lldb/scripts/LLDBWrapPython.cpp:54935:14: warning: explicitly assigning a variable of type 'void *' to itself [-Wself-assign]<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> clientdata = clientdata;<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"> ~~~~~~~~~~ ^ ~~~~~~~~~~<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal">4 warnings generated.<o:p></o:p></p>
</div>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</div>
</div>
</body>
</html>