Commit bf967fb3 authored by Muhammad Usama Anjum's avatar Muhammad Usama Anjum Committed by Shuah Khan
Browse files

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: default avatarMuhammad Usama Anjum <usama.anjum@collabora.com>
Signed-off-by: default avatarJohn Hubbard <jhubbard@nvidia.com>
Signed-off-by: default avatarShuah Khan <skhan@linuxfoundation.org>
parent 825658b7

tools/testing/selftests/x86/test_FISTTP.c

+4 −4
Original line number Diff line number Diff line
@@ -25,7 +25,7 @@ int test(void) 
	feclearexcept(FE_DIVBYZERO|FE_INEXACT|FE_INVALID|FE_OVERFLOW|FE_UNDERFLOW);

	asm volatile ("\n"

	"	fld1""\n"

	"	fisttp	res16""\n"

	"	fisttps	res16""\n"

	"	fld1""\n"

	"	fisttpl	res32""\n"

	"	fld1""\n"
@@ -45,7 +45,7 @@ int test(void) 
	feclearexcept(FE_DIVBYZERO|FE_INEXACT|FE_INVALID|FE_OVERFLOW|FE_UNDERFLOW);

	asm volatile ("\n"

	"	fldpi""\n"

	"	fisttp	res16""\n"

	"	fisttps	res16""\n"

	"	fldpi""\n"

	"	fisttpl	res32""\n"

	"	fldpi""\n"
@@ -66,7 +66,7 @@ int test(void) 
	asm volatile ("\n"

	"	fldpi""\n"

	"	fchs""\n"

	"	fisttp	res16""\n"

	"	fisttps	res16""\n"

	"	fldpi""\n"

	"	fchs""\n"

	"	fisttpl	res32""\n"
@@ -88,7 +88,7 @@ int test(void) 
	feclearexcept(FE_DIVBYZERO|FE_INEXACT|FE_INVALID|FE_OVERFLOW|FE_UNDERFLOW);

	asm volatile ("\n"

	"	fldln2""\n"

	"	fisttp	res16""\n"

	"	fisttps	res16""\n"

	"	fldln2""\n"

	"	fisttpl	res32""\n"

	"	fldln2""\n"