<html>
  <head>

    <meta http-equiv="content-type" content="text/html; charset=ISO-8859-1">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Hi,<br>
    <br>
    please review the following patch for 8033332.<br>
    <br>
    <b>Problem</b><br>
    The 32 bit version of
    TemplateInterpreterGenerator::generate_result_handler_for(..) does
    not handle the basic types T_LONG and T_VOID. Currently it does not
    fail because AbstractInterpreter::BasicType_as_index(..) returns the
    same index (4) for T_INT, T_LONG and T_VOID and therefore
    TemplateInterpreterGenerator::generate_all(..) invokes the method
    only once for T_INT. This can cause problems if the ordering of
    types changes.<br>
    <br>
    <b>Solution</b><br>
    Add missing case statements for T_LONG and T_VOID to
    generate_result_handler_for(..).<br>
    <br>
    <b>Testing</b><br>
    JPRT<br>
    <br>
    Bug: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8033332">https://bugs.openjdk.java.net/browse/JDK-8033332</a><br>
    Webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~thartmann/8033332/webrev.00/">http://cr.openjdk.java.net/~thartmann/8033332/webrev.00/</a><br>
    <br>
    Thanks,<br>
    Tobias<br>
  </body>
</html>