Skip to content
This repository was archived by the owner on Dec 22, 2021. It is now read-only.

[test] Add more literal tests for min/max/abs ops#154

Merged
tlively merged 2 commits intoWebAssembly:masterfrom
Honry:sync-wavm-234
Dec 12, 2019
Merged

[test] Add more literal tests for min/max/abs ops#154
tlively merged 2 commits intoWebAssembly:masterfrom
Honry:sync-wavm-234

Conversation

@Honry
Copy link
Contributor

@Honry Honry commented Dec 11, 2019

This PR covers tests for some float literal introduced in WebAssembly/spec#1069, specific for following ops:

f32x4.abs
f32x4.min
f32x4.max
f64x2.abs
f64x2.min
f64x2.max

These tests are pulled from WAVM/WAVM#234.

This PR covers tests for some float literal introduced in
WebAssembly/spec#1069

These tests are pulled from WAVM/WAVM#234
@Honry
Copy link
Contributor Author

Honry commented Dec 11, 2019

@tlively, @arunetm, PTAL, thanks!

return '-0x0p+0'
result = min(f1, f2)
if hex_form:
result = min(f1, f2)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This logic would be easier to follow if this was simply return min(f1, f2).hex(). Same below.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed, done.

@tlively tlively merged commit 303d8fc into WebAssembly:master Dec 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants