This is an archive of the discontinued LLVM Phabricator instance.

[ARM] Do not scale vext with a factor
ClosedPublic

Authored by jketema on Sep 18 2015, 10:24 AM.

Details

Summary

The vext pseudo-instruction takes the number of elements that need to be extracted, not the number of bytes. Hence, use the number of elements directly instead of scaling them with a factor.

Diff Detail

Event Timeline

jketema updated this revision to Diff 35103.Sep 18 2015, 10:24 AM
jketema retitled this revision from to [ARM] Do not scale vext with a factor.
jketema updated this object.
jketema added reviewers: sbaranga, rengolin.
jketema added a subscriber: llvm-commits.
sbaranga edited edge metadata.Sep 21 2015, 4:43 AM

I believe this is correct, thanks!

So can this be committed, or does someone else still need to look at this?

This revision was automatically updated to reflect the committed changes.