[PATCH] D82602: [SelectionDAG] don't split branch on logic-of-vector-compares

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 26 01:04:12 PDT 2020


RKSimon added a comment.

In D82602#2115689 <https://reviews.llvm.org/D82602#2115689>, @efriedma wrote:

> Would it make sense to treat this as an SLP issue?  i.e. we should transform `and(extractelement(x,0), extractelement(x,1))` to `vector.reduce.and(x)`?  If we're not extracting two elements from the same vector, the transform doesn't look as good.
>
> On AArch64, we probably want to use addp for the result of a `<2 x double>` compare, given the limited set of available operations on 64-bit integers, sure.  But I doubt it's really that relevant in practice.


we've tried several times to get 2 element vector support into SLP and its always blown up in our faces


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

https://reviews.llvm.org/D82602





More information about the llvm-commits mailing list