[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