[PATCH] D14082: [ARM] Expand ROTL and ROTR of vector value types

Charlie Turner via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 27 03:29:46 PDT 2015


chatur01 added a comment.

In http://reviews.llvm.org/D14082#275694, @RKSimon wrote:

> Charlie - thanks for dealing with this - I'm surprised it didn't turn up in the fuzz testing that I've been doing recently (the XOP rotation lowering definitely caught several times). I agree with Tim that making Expand the default is probably the way forward.


No worries Simon. Thanks for your improvements. I've landed this change in r251401.


http://reviews.llvm.org/D14082





More information about the llvm-commits mailing list