Skip to content

Commit bf967fb

Browse files
musamaanjumshuahkh
authored andcommitted
selftests: x86: test_FISTTP: use fisttps instead of ambiguous fisttp
Use fisttps instead of fisttp to specify correctly that the output variable is of size short. test_FISTTP.c:28:3: error: ambiguous instructions require an explicit suffix (could be 'fisttps', or 'fisttpl') 28 | " fisttp res16""\n" | ^ <inline asm>:3:2: note: instantiated into assembly here 3 | fisttp res16 | ^ ...followed by three more cases of the same warning for other lines. [jh: removed a bit of duplication from the warnings report, above, and fixed a typo in the title] Signed-off-by: Muhammad Usama Anjum <[email protected]> Signed-off-by: John Hubbard <[email protected]> Signed-off-by: Shuah Khan <[email protected]>
1 parent 825658b commit bf967fb

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

tools/testing/selftests/x86/test_FISTTP.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ int test(void)
2525
feclearexcept(FE_DIVBYZERO|FE_INEXACT|FE_INVALID|FE_OVERFLOW|FE_UNDERFLOW);
2626
asm volatile ("\n"
2727
" fld1""\n"
28-
" fisttp res16""\n"
28+
" fisttps res16""\n"
2929
" fld1""\n"
3030
" fisttpl res32""\n"
3131
" fld1""\n"
@@ -45,7 +45,7 @@ int test(void)
4545
feclearexcept(FE_DIVBYZERO|FE_INEXACT|FE_INVALID|FE_OVERFLOW|FE_UNDERFLOW);
4646
asm volatile ("\n"
4747
" fldpi""\n"
48-
" fisttp res16""\n"
48+
" fisttps res16""\n"
4949
" fldpi""\n"
5050
" fisttpl res32""\n"
5151
" fldpi""\n"
@@ -66,7 +66,7 @@ int test(void)
6666
asm volatile ("\n"
6767
" fldpi""\n"
6868
" fchs""\n"
69-
" fisttp res16""\n"
69+
" fisttps res16""\n"
7070
" fldpi""\n"
7171
" fchs""\n"
7272
" fisttpl res32""\n"
@@ -88,7 +88,7 @@ int test(void)
8888
feclearexcept(FE_DIVBYZERO|FE_INEXACT|FE_INVALID|FE_OVERFLOW|FE_UNDERFLOW);
8989
asm volatile ("\n"
9090
" fldln2""\n"
91-
" fisttp res16""\n"
91+
" fisttps res16""\n"
9292
" fldln2""\n"
9393
" fisttpl res32""\n"
9494
" fldln2""\n"

0 commit comments

Comments
 (0)