<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=koi8-r">
<style type="text/css" style="display:none"><!-- p { margin-top: 0px; margin-bottom: 0px; }--></style>
</head>
<body dir="ltr" style="font-size:12pt;color:#000000;background-color:#FFFFFF;font-family:Calibri,Arial,Helvetica,sans-serif;">
<p><span style="font-size: 12pt; color: rgb(33, 33, 33);">>>It probably better then form sugested by compiler</span><br>
</p>
<div style="color: rgb(33, 33, 33);">
<div>
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote"><span style="font-size: 12pt;"></span><span style="font-size: 12pt;">>>  if ((In<ELFT>::Common = createCommonSection<ELFT>()))</span></div>
<div class="gmail_quote"><span style="font-size: 12pt;">>></span><span style="font-size: 12pt;">    Add(In<ELFT>::Common);</span>
<div>> </div>
<div>>I don't know why I missed this during code review, but adding parentheses isn't a good way to fix the warning. You should use >more vertical space.</div>
<div>> </div>
<div>>  In<ELFT>::Common = createCommonSection<ELFT>();</div>
<div>>  if (In<ELFT>::Common)</div>
<div>>    ....<br>
</div>
<div><br>
</div>
<div>It was not on review, because there was no warning for me during development.<br>
</div>
<div>Here I was mean that when I fixed that warning after commit, compiler at first suggested me to add parantheses, but I did not do that (I knew we dont use this method) and used the same way as in your suggestion to fix the warning finally (https://reviews.llvm.org/rL298086).<br>
</div>
<div><br>
</div>
<div>George.<br>
</div>
</div>
</div>
</div>
</div>
</div>
</body>
</html>