[PATCH] D43172: [SelectionDAG][ARM][X86] Teach PromoteIntRes_SETCC to do a better job picking the result type for the setcc.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: test/CodeGen/X86/bitcast-and-setcc-512.ll:49
 ; SSE-NEXT:    # kill: def $al killed $al killed $eax
 ; SSE-NEXT:    retq
 ;
----------------
efriedma wrote:
> I guess the problem here is specifically that the new version doesn't use packssdw?
Yeah. I think we are only able to do it previously because the legalization process legalized the setcc several times and kept introducing truncates each time. With this patch we do less legalizations and introduce less intermediate truncates.


https://reviews.llvm.org/D43172