<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 20, 2017 at 9:07 AM, Bill Seurer <span dir="ltr"><<a href="mailto:seurer@linux.vnet.ibm.com" target="_blank">seurer@linux.vnet.ibm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 03/16/2017 04:38 PM, Kostya Serebryany wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Bill,<br>
<br>
* Has this been code-reviewed? In general, please do pre-commit code<br>
reviews for any code that is common between Power and other platforms.<br>
</blockquote>
<br></span>
This came from a gcc developer.  I submitted the patch here at his request.</blockquote><div><br></div><div>Please try to avoid this in future. The upstream is LLVM repo, not GCC repo, </div><div>so any code going into LLVM needs to be reviewed (at least if the committer is not a code owner)</div><div><br></div><div>--kcc  <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Even if (or: especially if) there are #ifdefs involved. (I hate #ifdefs)<br>
* Is there any chance to get rid of #ifdefs and replace them with if()?<br>
Or maybe just a separate implementation for Power?<br>
</blockquote>
<br></span>
The code is already #ifed for power but I will ask the developer.<span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
* do the existing tests cover the problems that this change solves?<br>
</blockquote>
<br></span>
The patch was to fix existing problems for 32 bit code that the existing tests cover.<span class="HOEnZb"><font color="#888888"><br>
-- <br>
<br>
-Bill Seurer<br>
<br>
</font></span></blockquote></div><br></div></div>