Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow Asm.js type coercions on simd builtin ret values #934

Merged

Conversation

arunetm-zz
Copy link
Contributor

@arunetm-zz arunetm-zz commented May 6, 2016

Allow simd builtin function return values to be coereced
to asm.js types as allowed by the spec.
http://asmjs.org/spec/latest/#unary-operators

Adding positive and negative tests.

Fixes #933

Allow simd builtin function return values to be coereced
to asm.js types as allowed by the spec.
http://asmjs.org/spec/latest/#unary-operators

Adding positive and negative tests.
@arunetm-zz
Copy link
Contributor Author

@abchatra @MikeHolman Please review.

@abchatra
Copy link
Contributor

abchatra commented May 6, 2016

@MikeHolman Can you please review this?

@MikeHolman
Copy link
Contributor

lgtm

@chakrabot chakrabot merged commit 0757418 into chakra-core:release/1.2 May 6, 2016
chakrabot pushed a commit that referenced this pull request May 6, 2016
Merge pull request #934 from arunetm:SimdBuiltinRetCoercion
Allow simd builtin function return values to be coereced
to asm.js types as allowed by the spec.
http://asmjs.org/spec/latest/#unary-operators

Adding positive and negative tests.

Fixes #933
chakrabot pushed a commit that referenced this pull request May 10, 2016
… ret values

Merge pull request #934 from arunetm:SimdBuiltinRetCoercion
Allow simd builtin function return values to be coereced
to asm.js types as allowed by the spec.
http://asmjs.org/spec/latest/#unary-operators

Adding positive and negative tests.

Fixes #933
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants