Skip to content

Make NaN handling more consistent.#772

Merged
gahaas merged 1 commit intoWebAssembly:masterfrom
gahaas:patch-5
Aug 23, 2016
Merged

Make NaN handling more consistent.#772
gahaas merged 1 commit intoWebAssembly:masterfrom
gahaas:patch-5

Conversation

@gahaas
Copy link
Contributor

@gahaas gahaas commented Aug 19, 2016

This PR makes the handling of NaNs in the promotion/demotion of floats more consistent to the handling of NaNs in arithmetic operations. Similar to #716, this PR requires that canonical NaNs have to be preserved by promotion/demotion. whereas the promotion/demotion of non-canonical NaNs can result in an arbitrary quiet NaN.

This PR makes the handling of NaNs in the  promotion/demotion of floats more consistent to the handling of NaNs in arithmetic operations. Similar to WebAssembly#716, this PR requires that canonical NaNs have to be preserved by promotion/demotion. whereas the promotion/demotion of non-canonical NaNs can result in an arbitrary quiet NaN.
@sunfishcode
Copy link
Member

lgtm. This is indeed the intention of #716.

@sunfishcode sunfishcode added this to the MVP milestone Aug 22, 2016
@gahaas gahaas merged commit 7ff98ec into WebAssembly:master Aug 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants