[LLVMdev] LLVM commit 410f38e01597120b41e406ec1cea69127463f9e5

[deadal nix]

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.


2014-07-08 16:26 GMT-07:00 Matt Arsenault <Matthew.Arsenault at amd.com>:

>  On 07/08/2014 03:20 PM, Matt Arsenault wrote:
>
> Alternatively maybe this should only be done if the setcc type is the same
> as the sext result?
>
> 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20140708/ec128dc0/attachment.html>