[PATCH] D67591: [ARM] A predicate cast of a predicate cast is a predicate cast

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 16 06:20:57 PDT 2019


dmgreen added a comment.

Yeah, it's a shame about the masked ld-st tests. They appear to be in a different order, causing the extra register usage. Seems like more of a scheduling problem. The tests in mve-pred-bitcast.ll and mve-pred-loadstore.ll are both smaller (and the masked load-store tests will eventually generate narrow/widen masked loads/stores.)

This seems like a fairly obvious simplification to me. It should in certain situations allow further simplification, so should in general be an improvement.


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

https://reviews.llvm.org/D67591





More information about the llvm-commits mailing list