Subzero. ARM32. Vector casts.

Subzero. ARM32. Vector casts.

This CL un-scalarizes all vector casts operations in Subzero.

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

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

[John, 2016-04-14 13:48:55]

Description was changed from

==========
Subzero. ARM32. i1 Vector Zext and Sext.

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

to

==========
Subzero. ARM32. i1 Vector Zext and Sext.

This CL also unscalarizes Vector truncs. Because of the way vector Zext and Sext
were implemented, vector truncs are trivially lowered to an assignment to the
narrower (i1) type.

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

[John, 2016-04-14 15:03:13]

Description was changed from

==========
Subzero. ARM32. i1 Vector Zext and Sext.

This CL also unscalarizes Vector truncs. Because of the way vector Zext and Sext
were implemented, vector truncs are trivially lowered to an assignment to the
narrower (i1) type.

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

to

==========
Subzero. ARM32. Vector casts.

This CL un-scalarizes all vector casts operations in Subzero.

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

[John, 2016-04-14 15:05:32]

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

[Eric Holk, 2016-04-14 17:40:39]

lgtm

[John, 2016-04-14 18:51:41]

Description was changed from

==========
Subzero. ARM32. Vector casts.

This CL un-scalarizes all vector casts operations in Subzero.

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

to

==========
Subzero. ARM32. Vector casts.

This CL un-scalarizes all vector casts operations in Subzero.

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

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

[John, 2016-04-14 18:51:42]

Committed patchset #5 (id:80001) manually as
e88c7deb72049a08ecad8bb8f9ed3696c9415b0e (presubmit successful).

[Jim Stichnoth, 2016-04-14 21:17:16]

lgtm

https://codereview.chromium.org/1878943009/diff/80001/tests_lit/assembler/arm...
File tests_lit/assembler/arm32/vcvt.f32.s32.ll (right):

https://codereview.chromium.org/1878943009/diff/80001/tests_lit/assembler/arm...
tests_lit/assembler/arm32/vcvt.f32.s32.ll:49: ; IASM-NOT:    vcvt.f32.s32
It might be "safer" to check against just "vcvt" like above.

[John, 2016-04-14 21:22:11]

https://codereview.chromium.org/1878943009/diff/80001/tests_lit/assembler/arm...
File tests_lit/assembler/arm32/vcvt.f32.s32.ll (right):

https://codereview.chromium.org/1878943009/diff/80001/tests_lit/assembler/arm...
tests_lit/assembler/arm32/vcvt.f32.s32.ll:49: ; IASM-NOT:    vcvt.f32.s32
On 2016/04/14 21:17:16, stichnot wrote:
> It might be "safer" to check against just "vcvt" like above.

Done (in the next CL.)