Add CLZ instruction to the ARM integrated assembler.

Add CLZ instruction to the ARM integrated assembler.

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=65f80d72389d036abaa3714ca18d95bf59ebb1fa

[Karl, 2015-12-11 23:23:47]

Description was changed from

==========
Add CLZ instruction to the ARM integrated assembler.

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

to

==========
Add CLZ instruction to the ARM integrated assembler.

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

[Karl, 2015-12-11 23:23:47]

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

[Jim Stichnoth, 2015-12-11 23:44:34]

lgtm

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

https://codereview.chromium.org/1521133002/diff/40001/tests_lit/assembler/arm...
tests_lit/assembler/arm32/clz.ll:3: ; NOTE: We use -Om1 to get rid of memory
stores.
All the other .ll files with this comment say the -O2 is used to get rid of
memory stores.  What's the real reason for -Om1 here?

I assume that as a general principle, Om1 is used to maximize stability of test
output, so maybe this comment just gets deleted?

[Karl, 2015-12-17 16:02:21]

Description was changed from

==========
Add CLZ instruction to the ARM integrated assembler.

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

to

==========
Add CLZ instruction to the ARM integrated assembler.

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-17 16:02:24]

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

[Karl, 2015-12-17 16:02:41]

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

https://codereview.chromium.org/1521133002/diff/40001/tests_lit/assembler/arm...
tests_lit/assembler/arm32/clz.ll:3: ; NOTE: We use -Om1 to get rid of memory
stores.
On 2015/12/11 23:44:34, stichnot wrote:
> All the other .ll files with this comment say the -O2 is used to get rid of
> memory stores.  What's the real reason for -Om1 here?
> 
> I assume that as a general principle, Om1 is used to maximize stability of
test
> output, so maybe this comment just gets deleted?

Yes, I wanted to maximize stability in this case. Removing comment.