<div dir="ltr"><div>Hi Eli,</div><div><br></div><div>I create a patch at <a href="https://reviews.llvm.org/D31488">https://reviews.llvm.org/D31488</a></div><div><br></div><div>Thanks</div><div>Hongbin</div><div> </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 29, 2017 at 3:18 PM, Friedman, Eli <span dir="ltr"><<a href="mailto:efriedma@codeaurora.org" target="_blank">efriedma@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 bgcolor="#FFFFFF" text="#000000"><span class="">
    <div class="m_-8155527773100066718moz-cite-prefix">On 3/29/2017 3:00 PM, Hongbin Zheng
      wrote:<br>
    </div>
    <blockquote type="cite">
      <div dir="ltr"><br>
        <div class="gmail_extra"><br>
          <div class="gmail_quote">On Wed, Mar 29, 2017 at 2:15 PM,
            Friedman, Eli <span dir="ltr"><<a href="mailto:efriedma@codeaurora.org" target="_blank">efriedma@codeaurora.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 class="m_-8155527773100066718gmail-">On
                3/29/2017 1:05 PM, Hongbin Zheng wrote:<br>
                <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
                  Hi Eli,<br>
                  <br>
                  Thanks. Do you mean ideally we should extend
                  SimplifyIndVar to do the sdiv->udiv replacement?<br>
                </blockquote>
                <br>
              </span>
              I haven't really looked into it closely, but it seems to
              make sense.</blockquote>
            <div>Ok.</div>
            <div><br>
            </div>
            <div>Once I extend SimplifyIndVar, e.g. the
               simplifyUsersOfIV function. How I can add a regression
              test for the extension? simplifyUsersOfIV  is not a pass.</div>
          </div>
        </div>
      </div>
    </blockquote>
    <br></span>
    It gets called by -indvars.<span class=""><br>
    <br>
    -Eli<br>
    <pre class="m_-8155527773100066718moz-signature" cols="72">-- 
Employee of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project</pre>
  </span></div>

</blockquote></div><br></div>