<html>

    <head>

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

    </head>

    <body><span class="vcard"><a class="email" href="mailto:michael.m.kuperstein@intel.com" title="Michael Kuperstein <michael.m.kuperstein@intel.com>"> <span class="fn">Michael Kuperstein</span></a>

</span> changed

              <a class="bz_bug_link 

          bz_status_REOPENED "

   title="REOPENED --- - Regression due to GMR changes in r242512"

   href="https://llvm.org/bugs/show_bug.cgi?id=24288">bug 24288</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>RESOLVED

           </td>

           <td>REOPENED

           </td>

         </tr>

         <tr>

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

           <td>FIXED

           </td>

           <td>---

           </td>

         </tr></table>

      <p>

        <div>

            <b><a class="bz_bug_link 

          bz_status_REOPENED "

   title="REOPENED --- - Regression due to GMR changes in r242512"

   href="https://llvm.org/bugs/show_bug.cgi?id=24288#c8">Comment # 8</a>

              on <a class="bz_bug_link 

          bz_status_REOPENED "

   title="REOPENED --- - Regression due to GMR changes in r242512"

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

              from <span class="vcard"><a class="email" href="mailto:michael.m.kuperstein@intel.com" title="Michael Kuperstein <michael.m.kuperstein@intel.com>"> <span class="fn">Michael Kuperstein</span></a>

</span></b>

        <pre>Some of the regressions in eembc were fixed, but others still remain.

There are two remaining issues I can see:

1) Easy: isNonEscapingGlobalNoAlias(), for the "load from global variable"

case, does not call GetUnderlyingObject() on the pointer operand. This means it

misses cases where the load is from a GEP/BitCast ConstantExpr based on a GV.

2) Harder: this is tangentially related to PR22475. The canonicalization from

r226781 produces inttoptr casts that isNonEscapingGlobalNoAlias() can't (and

probably shouldn't try to) look through. I think the fix for this should be on

the InstCombine end, not GMR.</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>