<html>

    <head>

      <base href="https://llvm.org/bugs/" />

    </head>

    <body><span class="vcard"><a class="email" href="mailto:spatel+llvm@rotateright.com" title="Sanjay Patel <spatel+llvm@rotateright.com>"> <span class="fn">Sanjay Patel</span></a>

</span> changed

              <a class="bz_bug_link 

          bz_status_RESOLVED  bz_closed"

   title="RESOLVED FIXED - sign extensions cause suboptimal pointer arithmetic?"

   href="https://llvm.org/bugs/show_bug.cgi?id=20134">bug 20134</a>

        <br>

             <table border="1" cellspacing="0" cellpadding="8">

          <tr>

            <th>What</th>

            <th>Removed</th>

            <th>Added</th>

          </tr>

         <tr>

           <td style="text-align:right;">Status</td>

           <td>ASSIGNED

           </td>

           <td>RESOLVED

           </td>

         </tr>

         <tr>

           <td style="text-align:right;">Resolution</td>

           <td>---

           </td>

           <td>FIXED

           </td>

         </tr></table>

      <p>

        <div>

            <b><a class="bz_bug_link 

          bz_status_RESOLVED  bz_closed"

   title="RESOLVED FIXED - sign extensions cause suboptimal pointer arithmetic?"

   href="https://llvm.org/bugs/show_bug.cgi?id=20134#c16">Comment # 16</a>

              on <a class="bz_bug_link 

          bz_status_RESOLVED  bz_closed"

   title="RESOLVED FIXED - sign extensions cause suboptimal pointer arithmetic?"

   href="https://llvm.org/bugs/show_bug.cgi?id=20134">bug 20134</a>

              from <span class="vcard"><a class="email" href="mailto:spatel+llvm@rotateright.com" title="Sanjay Patel <spatel+llvm@rotateright.com>"> <span class="fn">Sanjay Patel</span></a>

</span></b>

        <pre>Thanks for working on this, Duncan. 

For the record, I did review your patch and tried all kinds of related things (

details in <a href="http://reviews.llvm.org/D13757">http://reviews.llvm.org/D13757</a> ), but I settled on an x86 DAG

solution because it caught a lot of x86-specific cases with a simple patch. 

But in the long-term, the consensus is that it would be better to improve

CodeGenPrepare or generalize AddressTypePromotion to do more of these

transforms.

Fix committed here:

<a href="http://reviews.llvm.org/rL250560">http://reviews.llvm.org/rL250560</a></pre>

        </div>

      </p>

      <hr>

      <span>You are receiving this mail because:</span>

      <ul>

          <li>You are on the CC list for the bug.</li>

      </ul>

    </body>

</html>