commit 33dedfbd7d9599e13d7e9efc0869920ee2d4732b
author Keith Packard <keithp@keithp.com> Sat May 20 17:29:23 2023 -0700
committer Anas Nashif <anas.nashif@intel.com> Fri May 26 09:55:52 2023 -0400
tree ed759ca422249dd9b35a2f7c7dc21e764d77776b
parent 2c7507036050804f207c0604073f38798bc1a5fd

tests/sprintf: Test floating printf on soft-fp targets too

There's no reason to limit testing floating point printf to platforms with
an FPU; neither the minimal C library nor picolibc even use floating point
instructions for printf. And even if they did, the toolchain should have
soft float support.

However, we do need to restrict picolibc testing to configurations with
floating point printf enabled.

Signed-off-by: Keith Packard <keithp@keithp.com>

tests/lib/sprintf/src/main.c

diff --git a/tests/lib/sprintf/src/main.c b/tests/lib/sprintf/src/main.c
index b36c909..19859b0 100644
--- a/tests/lib/sprintf/src/main.c
+++ b/tests/lib/sprintf/src/main.c
@@ -42,6 +42,9 @@
 #define IS_MINIMAL_LIBC_NOFP (IS_ENABLED(CONFIG_MINIMAL_LIBC) \
 	      && !IS_ENABLED(CONFIG_CBPRINTF_FP_SUPPORT))
 
+#define IS_PICOLIBC_NOFP (IS_ENABLED(CONFIG_PICOLIBC) \
+	      && !IS_ENABLED(CONFIG_PICOLIBC_IO_FLOAT))
+
 /*
  * A really long string (330 characters + NULL).
  * The underlying sprintf() architecture will truncate it.
@@ -106,11 +109,11 @@
 	/* Conversion not supported with minimal_libc without
 	 * CBPRINTF_FP_SUPPORT.
 	 *
-	 * Conversion not supported without FPU except on native POSIX.
+	 * Conversion not supported with picolibc without
+	 * PICOLIBC_IO_FLOAT
+	 *
 	 */
-	if (IS_MINIMAL_LIBC_NOFP
-	    || !(IS_ENABLED(CONFIG_FPU)
-		 || IS_ENABLED(CONFIG_BOARD_NATIVE_POSIX))) {
+	if (IS_MINIMAL_LIBC_NOFP || IS_PICOLIBC_NOFP) {
 		ztest_test_skip();
 		return;
 	}