[PATCH] D58521: [DAGCombiner] allow truncation of binops after legalization if desirable
Ulrich Weigand via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
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
More information about the llvm-commits
mailing list