<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 20 November 2013 19:39, Weiming Zhao <span dir="ltr"><<a href="mailto:weimingz@codeaurora.org" target="_blank">weimingz@codeaurora.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-US" link="blue" vlink="purple"><div><p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">Ping?<u></u><u></u></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1f497d">The patch won’t affect other targets</span></p></div></div></blockquote><div></div></div><br></div><div class="gmail_extra">
Hi Weiming,</div><div class="gmail_extra"><br></div><div class="gmail_extra">Regarding the original patch, I think hijacking the current function with another, similar function, will make the code worse than it already is to understand DivRem lowering. That confusion will affect all targets, and that was the major concern with my first patch, which changed in a slightly more intrusive, but also more clear way.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">It'd be good if you could try to add these changes to the ARM back-end (like I did), and upon failure (like I had), we'll have more reasons to try a more generic approach (or refactor the generic approach to add this in the ARM back-end without the same problems, though the problems I had were deep inside SelectionDAG type legalization). It is also possible that you will succeed, in which case we'll be able to get the patch across. I doubt anyone now will want to re-factor the SelectionDAG type legalization...</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Only after failing to get it through the ARM back-end that I'd re-consider the generic approach. This time, involving as many people as possible to get this over with once and for all. </div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Don't take this personally, it's just an advice that this patch will not pass review as it is, given the context.</div><div class="gmail_extra"><br></div>
<div class="gmail_extra">cheers,</div><div class="gmail_extra">--renato</div><div class="gmail_extra"><br></div></div>