<html>

  <head>

    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">

  </head>

  <body bgcolor="#FFFFFF" text="#000000">

    <div class="moz-cite-prefix">On 1/23/2017 11:47 PM, Jonas Paulsson

      wrote:<br>

    </div>

    <blockquote

      cite="mid:da7b98cf-a4a5-18f1-5d7c-5ae3e1eabb8b@linux.vnet.ibm.com"

      type="cite">

      <blockquote type="cite" style="color: #000000;"><br>

      </blockquote>

      Thanks Eli - that seemed to do the trick! <span

        class="moz-smiley-s1" title=":-)"><span>:-)</span></span>

      <br>

      <br>

      I just added a few lines to convert the node to SIGN/ZERO

      _EXTEND_VECTOR_INREG - see attachment.

      <br>

      <br>

      I am thinking that the WidenVT should be legal, and since it seems

      GetWidenedVector() is supposed to always return a legal vector, I

      have neither added a check for legality of InOp. All regression

      tests pass. </blockquote>

    <br>

    <p>Better to use "DAG.getSignExtendVectorInReg(..." rather than

      "DAG.getNode(ISD::SIGN_EXTEND_VECTOR_INREG, ...", probably.  (You

      might also be hiding some problems; it looks like the assertions

      which verify the input and output types are in

      getSignExtendVectorInReg rather than getNode.)<br>

    </p>

    <p>Are you sure "InOp" is the right input to the operation? 

      SIGN_EXTEND_VECTOR_INREG expects that the input and the output are

      the same size, but you aren't doing anything to ensure that.</p>

    <p>-Eli<br>

    </p>

    <pre class="moz-signature" cols="72">-- 

Employee of Qualcomm Innovation Center, Inc.

Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project</pre>

  </body>

</html>