[PATCH] D58521: [DAGCombiner] allow truncation of binops after legalization if desirable

Tue Feb 26 02:27:55 PST 2019

uweigand added a comment.

In D58521#1409255 <https://reviews.llvm.org/D58521#1409255>, @jonpa wrote:

> I tried this patch on SystemZ / SPEC, and as before this seems to have a relatively very minor impact on the number of files changed (7), and on the performance (seemingly unaffected).
>
> I think the SystemZ test looks good, but I leave the final approval to Uli as usual.

Ah, the SystemZ test changes certainly LGTM, and would be a welcome improvement ...

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58521/new/

https://reviews.llvm.org/D58521