[PATCH] D11123: ARM: Fix cttz expansion on vector types.
Renato Golin
renato.golin at linaro.org
Mon Jul 13 08:23:19 PDT 2015
rengolin accepted this revision.
rengolin added a comment.
This revision is now accepted and ready to land.
Hi Logan, looks good, thanks!
The cases where you didn't add CHECKs, was it because it didn't use vector instructions? I think it should be fine to check for scalar instructions, like the others, at least to check that the instructions you were expecting got lowered.
Otherwise, LGTM. Thanks!
http://reviews.llvm.org/D11123
More information about the llvm-commits
mailing list