<html>
    <head>
      <base href="https://bugs.llvm.org/">
    </head>
    <body><table border="1" cellspacing="0" cellpadding="8">
        <tr>
          <th>Bug ID</th>
          <td><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Backport SVN r309744 to 5.0"
   href="https://bugs.llvm.org/show_bug.cgi?id=34025">34025</a>
          </td>
        </tr>

        <tr>
          <th>Summary</th>
          <td>Backport SVN r309744 to 5.0
          </td>
        </tr>

        <tr>
          <th>Product</th>
          <td>libraries
          </td>
        </tr>

        <tr>
          <th>Version</th>
          <td>5.0
          </td>
        </tr>

        <tr>
          <th>Hardware</th>
          <td>PC
          </td>
        </tr>

        <tr>
          <th>OS</th>
          <td>All
          </td>
        </tr>

        <tr>
          <th>Status</th>
          <td>NEW
          </td>
        </tr>

        <tr>
          <th>Severity</th>
          <td>enhancement
          </td>
        </tr>

        <tr>
          <th>Priority</th>
          <td>P
          </td>
        </tr>

        <tr>
          <th>Component</th>
          <td>Backend: AArch64
          </td>
        </tr>

        <tr>
          <th>Assignee</th>
          <td>unassignedbugs@nondot.org
          </td>
        </tr>

        <tr>
          <th>Reporter</th>
          <td>martin@martin.st
          </td>
        </tr>

        <tr>
          <th>CC</th>
          <td>llvm-bugs@lists.llvm.org
          </td>
        </tr></table>
      <p>
        <div>
        <pre>Please backport r309744 ([AArch64] Rewrite stack frame handling for win64
vararg functions) to 5.0. This fixes another issue with the new win/arm64
calling convention - similar to the one in PR33920 that already was backported
(thanks!).

The rationale is the same as in that one: The support for targeting win/arm64
itself is still incomplete, but the calling convention handling in itself is
getting complete, and can be used by Wine for clang >= 5.0. (The code already
in 5.0 works for Wine for the codepaths I've tested so far, but this makes it
more correct, in case other, so far untested, functions are compiled
differently.)

This carries no regression risk as it only touches code in new codepaths that
aren't used before. Even though it renames a variable, it doesn't have any
practical effect on other targets than windows.

One of the touched files, test/CodeGen/AArch64/win64_vararg.ll, got another
change inbetween, in r309015 (just an added testcase) - the one to backport
should apply cleanly both with and without the updated test (and backporting
the test shouldn't hurt either if that's an option).</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>