Skip to content

Remove exact nan-pattern tests #381

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

Merged
merged 1 commit into from
Nov 22, 2016
Merged

Conversation

gahaas
Copy link
Collaborator

@gahaas gahaas commented Nov 10, 2016

This PR removes exact some remaining exact nan-pattern tests because they became obsolete in WebAssembly/design#716. Similar tests without bit pattern matching exist in f32.wast and f64.wast.

@rossberg
Copy link
Member

rossberg commented Nov 10, 2016

LGTM, but @sunfishcode should sign off as well.

@gahaas
Copy link
Collaborator Author

gahaas commented Nov 16, 2016

@sunfishcode Could you please take a look?

@sunfishcode
Copy link
Member

lgtm. As I commented in #286, these are important cases to cover, but for now we lack a way to effectively test them.

@lukewagner
Copy link
Member

@bnjbvr Did we test these using the magic test mode so that the engine spits out the literal bit contents?

@bnjbvr
Copy link
Member

bnjbvr commented Nov 21, 2016

@lukewagner Yes indeed; any spec test that uses equality with a precise NaN pattern checks against that same pattern spit out as a literal integer. All the spec tests are run under the magic test mode, in Spidermonkey CI.

@gahaas gahaas merged commit d0585b1 into WebAssembly:master Nov 22, 2016
@gahaas gahaas deleted the fix-float-misc branch November 22, 2016 08:24
ngzhian pushed a commit to ngzhian/spec that referenced this pull request Nov 4, 2021
ngzhian added a commit to ngzhian/spec that referenced this pull request Nov 4, 2021
These instructions were added in WebAssembly#381 and WebAssembly#411 respectively.

The binary opcodes for these are still not finalized, I'm using what V8
is using for now.
raoxiaojia pushed a commit to WasmCert/spec that referenced this pull request Apr 29, 2025
Exact nan-pattern tests got obsolete with WebAssembly/design#716 (WebAssembly#381)
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