From 7f30b6c4c0af98642a8f22cbd4e61ff688f9e9fe Mon Sep 17 00:00:00 2001 From: Brendan Dahl Date: Thu, 3 Oct 2024 16:34:29 -0700 Subject: [FP16] Fix comment on f16x8.convert_i16x8_s test. (#6985) 0x7800 as FP16 is 32,768 not 32,767 as I had in the comment. I also added another comment to explain the somewhat confusing behavior. --- test/spec/f16.wast | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'test/spec') diff --git a/test/spec/f16.wast b/test/spec/f16.wast index b495fe2b6..60bcc74ba 100644 --- a/test/spec/f16.wast +++ b/test/spec/f16.wast @@ -238,9 +238,9 @@ (v128.const i16x8 0x5140 0x7e00 0x7c00 0xfc00 0x7bff 0xfbff 0 0)) (v128.const i16x8 42 0 65535 0 65504 0 0 0)) (assert_return (invoke "f16x8.convert_i16x8_s" - ;; + ;; 32767 is not representable as a whole integer in FP16, so it becomes 32768. (v128.const i16x8 0 1 -1 32767 -32768 0 0 0)) - ;; 0 1 -1 32767 -32768 0 0 0 + ;; 0 1 -1 32768 -32768 0 0 0 (v128.const i16x8 0 0x3c00 0xbc00 0x7800 0xf800 0 0 0)) (assert_return (invoke "f16x8.convert_i16x8_u" ;; Unlike f32/64, f16 can't represent the full 2^16 integer range so 2^16 becomes infinity. -- cgit v1.2.3