[PATCH] D76709: [Target][ARM] Adding MVE VPT Optimisation Pass

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 2 01:36:00 PDT 2020


dmgreen added a comment.

Looking good.

Can you add some extra ll tests, made from intrinsics that show different blocks being created, but testing the entire backend. All kinds of things from simple to complicated if you can. A good collection of VCMP, conditional VCMP, other conditional instructions and VPNOT's in different positions. Then add --verify-machine-instr to it. It doesn't matter if they are not all optimal yet, they will show what is working well and what isn't yet.


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

https://reviews.llvm.org/D76709





More information about the llvm-commits mailing list