<div dir="ltr">The proposed patch would work for me. The combine make sense for me 100%
 of the time as my target can have different type for the setcc and the 
selected values. So if it is not in DAGCombiner, I'm going to end up 
doing it one way or another in the backend.</div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-07-08 16:26 GMT-07:00 Matt Arsenault <span dir="ltr"><<a href="mailto:Matthew.Arsenault@amd.com" target="_blank">Matthew.Arsenault@amd.com</a>></span>:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div text="#000000" bgcolor="#FFFFFF"><div class="">
    <div>On 07/08/2014 03:20 PM, Matt Arsenault
      wrote:<br>
    </div>
    <blockquote type="cite">
      
      Alternatively maybe this should only be done if the setcc type is
      the same as the sext result?<br>
    </blockquote></div>
    I think we should actually do this. If you need to convert the setcc
    result after, you aren't really gaining anything by doing this
    transformation<br>
  </div>

</blockquote></div><br></div>