[PATCH] D54640: [DAGCombiner] narrow truncated binops

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 19 10:07:21 PST 2018


spatel added a comment.

In https://reviews.llvm.org/D54640#1303043, @RKSimon wrote:

> X86's combineTruncatedArithmetic does something similar - can we merge them at all?


Yes, that should be possible. This patch has a TODO about supporting vectors, and that code has a TODO about supporting scalars (it is exclusively working on vectors). So they are completely independent for the moment, but I'll take that as a follow-up.


https://reviews.llvm.org/D54640





More information about the llvm-commits mailing list