Implement LSR instructions for the integrated ARM assembler.

Implement LSR instructions for the integrated ARM assembler.

Also factors out the body of method lsl() to emitShift(), so
that all forms of shift instructions can use the same code.

BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=4334
R=stichnot@chromium.org

Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=4ddce702d86894de5d7e60ed0178df86ed2e7dd1

[Karl, 2015-12-04 22:59:44]

Description was changed from

==========
Implement LSR instructions for the integrated ARM assembler.

Also factors out the body of method lsl() to emitShift(), so
that all forms of shift instructions can use the same code.

BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=4334
==========

to

==========
Implement LSR instructions for the integrated ARM assembler.

Also factors out the body of method lsl() to emitShift(), so
that all forms of shift instructions can use the same code.

BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=4334
==========

[Karl, 2015-12-04 22:59:44]

kschimpf@google.com changed reviewers:
+ jpp@chromium.org, sehr@chromium.org, stichnot@chromium.org

[Jim Stichnoth, 2015-12-04 23:39:12]

lgtm

https://codereview.chromium.org/1501073002/diff/40001/tests_lit/assembler/arm...
File tests_lit/assembler/arm32/lsr.ll (right):

https://codereview.chromium.org/1501073002/diff/40001/tests_lit/assembler/arm...
tests_lit/assembler/arm32/lsr.ll:32: %shl = lshr i32 %a, 23
Maybe you want to rename this var?

[Karl, 2015-12-07 18:43:37]

Description was changed from

==========
Implement LSR instructions for the integrated ARM assembler.

Also factors out the body of method lsl() to emitShift(), so
that all forms of shift instructions can use the same code.

BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=4334
==========

to

==========
Implement LSR instructions for the integrated ARM assembler.

Also factors out the body of method lsl() to emitShift(), so
that all forms of shift instructions can use the same code.

BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=4334
R=stichnot@chromium.org

Committed:
https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a...
==========

[Karl, 2015-12-07 18:43:38]

Committed patchset #4 (id:60001) manually as
4ddce702d86894de5d7e60ed0178df86ed2e7dd1 (presubmit successful).

[Karl, 2015-12-07 18:44:00]

https://codereview.chromium.org/1501073002/diff/40001/tests_lit/assembler/arm...
File tests_lit/assembler/arm32/lsr.ll (right):

https://codereview.chromium.org/1501073002/diff/40001/tests_lit/assembler/arm...
tests_lit/assembler/arm32/lsr.ll:32: %shl = lshr i32 %a, 23
On 2015/12/04 23:39:12, stichnot wrote:
> Maybe you want to rename this var?

Done.