<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Mar 17, 2017 at 6:28 AM, George Rimar via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span>>I am now getting:<br>
><br>
>/home/fedora/llvm/llvm/tools/<wbr>lld/ELF/Writer.cpp:361:24: warning: using the result of an assignment as a condition without >parentheses [-Wparentheses]<br>
><br>
>Cheers,<br>
>Rafael<br>
<br>
</span>Fixed in r298082<br>
<br>
It probably better then form sugested by compiler:<br>
  if ((In<ELFT>::Common = createCommonSection<ELFT>()))<br>
    Add(In<ELFT>::Common);</blockquote><div><br></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><br></div><div>  In<ELFT>::Common = createCommonSection<ELFT>();</div><div>  if (In<ELFT>::Common)</div><div>    ....</div></div></div></div>