[PATCH] D56118: [ARM]: Add optimized NEON uint64x2_t multiply routine.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added subscribers: spatel, RKSimon, craig.topper.
craig.topper added inline comments.


================
Comment at: test/CodeGen/ARM/vmul.ll:69
+;CHECK: vld1.64         {d20, d21}, [r0]
+;CHECK; vand            q8, q10, q8
+;CHECK: vmovn.i64       d18, q9
----------------
efriedma wrote:
> DAGCombine should be able to catch the redundant AND... but it looks like DAGCombiner::visitTRUNCATE doesn't try to handle demanded bits for vectors.  (I guess it didn't get updated when other operations got support for vector operands?)
PR39689 mentions this is disabled for vectors. Maybe @rksimon or @spatel are working on it?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56118