<html>
    <head>
      <base href="https://bugs.llvm.org/">
    </head>
    <body><span class="vcard"><a class="email" href="mailto:mikael.holmen@ericsson.com" title="Mikael Holmén <mikael.holmen@ericsson.com>"> <span class="fn">Mikael Holmén</span></a>
</span> changed
          <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - Missing vectorization of loop due to load late in the loop (unsupported first order recurrence)"
   href="https://bugs.llvm.org/show_bug.cgi?id=45823">bug 45823</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;">Resolution</td>
           <td>---
           </td>
           <td>FIXED
           </td>
         </tr>

         <tr>
           <td style="text-align:right;">Status</td>
           <td>NEW
           </td>
           <td>RESOLVED
           </td>
         </tr></table>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - Missing vectorization of loop due to load late in the loop (unsupported first order recurrence)"
   href="https://bugs.llvm.org/show_bug.cgi?id=45823#c2">Comment # 2</a>
              on <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - Missing vectorization of loop due to load late in the loop (unsupported first order recurrence)"
   href="https://bugs.llvm.org/show_bug.cgi?id=45823">bug 45823</a>
              from <span class="vcard"><a class="email" href="mailto:mikael.holmen@ericsson.com" title="Mikael Holmén <mikael.holmen@ericsson.com>"> <span class="fn">Mikael Holmén</span></a>
</span></b>
        <pre>It seems that with commit aa00b1d76364:

    [LV] Try to sink users recursively for first-order recurrences.

    Update isFirstOrderRecurrence to  explore all uses of a recurrence phi
    and check if we can sink them. If there are multiple users to sink, they
    are all mapped to the previous instruction.

    Fixes PR44286 (and another PR or two).

    Reviewed By: Ayal

    Differential Revision: <a href="https://reviews.llvm.org/D84951">https://reviews.llvm.org/D84951</a>

the vectorizer doesn't bail out on the loop anymore.
Now we instead get

LV: Vectorization is possible but not beneficial.

and if adding e.g.
 -force-vector-width=2
the loop indeed gets vectorized.

Thanks!</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>