add fixmes for disagreement in the cost of JavaScript ToInt32 without fcvttgw
authorJacob Lifshay <programmerjake@gmail.com>
Wed, 22 Mar 2023 22:13:24 +0000 (15:13 -0700)
committerJacob Lifshay <programmerjake@gmail.com>
Wed, 22 Mar 2023 22:13:24 +0000 (15:13 -0700)
openpower/sv.mdwn
openpower/sv/int_fp_mv.mdwn
openpower/sv/int_fp_mv/appendix.mdwn
openpower/sv/rfc/ls006.mdwn

index 1e9749ec057951fff8bf973817f37113c0a2e1c3..d9b5b45b90ef0cb9b80ca3db819e5af3c746b588 100644 (file)
@@ -223,7 +223,7 @@ Scalable Vector binaries more efficient, such
 as the crweird group.  Others are to bring the Scalar Power ISA
 up-to-date within specific workloads,
 such as a JavaScript Rounding instruction
-(which saves 35 instructions including 5 branches). None of them are strictly
+(which saves 35 instructions including 5 branches) (FIXME: disagrees with int_fp_mv and int_fp_mv/appendix). None of them are strictly
 necessary but performance and power consumption may be (or, is already)
 compromised
 in certain workloads and use-cases without them.
index 114ad9e3423654eed8f202b06e06cb9c5550edc0..4e8f40bd93843b6b451763c1ee0df349bcc3f9d6 100644 (file)
@@ -89,6 +89,7 @@ well-suited for common/important conversion sequences:
 The assembly listings in the [[int_fp_mv/appendix]] show how costly
 some of these language-specific conversions are: JavaScript, the
 worst case, is 32 scalar instructions including seven branch instructions.
+(FIXME: disagrees with ls006 and sv.mdwn)
 
 # Proposed New Scalar Instructions
 
index 02a002e84f767fbd72bcaeadbd737f6868412ba3..57eaf9d706046fb5205c53de640c43802f8fa461 100644 (file)
@@ -271,6 +271,8 @@ inline Target bitwise_cast(Src v) {
 
 ### 64-bit float -> 32-bit signed integer
 
+(FIXME: disagrees with ls006 and sv.mdwn)
+
 ```
 toInt32(double):
         stfd 1,-16(1)
index f92515079a00999b94c2fb958335747182632994..4bc744b0d69f62a0082a89a653a26fbe69012b07 100644 (file)
@@ -72,7 +72,7 @@ implements each variant.
 
 * These instructions are present in many other ISAs.
 * JavaScript rounding as one instruction saves 35 instructions including
-  six branches.
+  six branches. (FIXME: disagrees with int_fp_mv and int_fp_mv/appendix)
 
 **Changes**