Replace small C lib with std C lib if not supported for a given target

Author: hugueskamba

tools/build_api.py

@@ -48,6 +48,7 @@
 from .libraries import Library
 from .toolchains import TOOLCHAIN_CLASSES, TOOLCHAIN_PATHS
 from .toolchains.arm import UARM_TOOLCHAIN_WARNING
+from .toolchains.mbed_toolchain import is_std_c_lib_supported_instead_small_c_lib
 from .config import Config
 
 RELEASE_VERSIONS = ['2', '5']
@@ -205,6 +206,7 @@ def get_toolchain_name(target, toolchain_name):
 
     return toolchain_name
 
+
 def find_valid_toolchain(target, toolchain):
     """Given a target and toolchain, get the names for the appropriate
     toolchain to use. The environment is also checked to see if the corresponding
@@ -250,6 +252,15 @@ def find_valid_toolchain(target, toolchain):
                 and "uARM" in {toolchain_name, target.default_toolchain}
             ):
                 end_warnings.append(UARM_TOOLCHAIN_WARNING)
+
+            if is_std_c_lib_supported_instead_small_c_lib(target, toolchain):
+                warning = (
+                    "Warning: We noticed that you are using target.c_lib set to small.\n"
+                    "As the {} target does not support a small C library for the {} toolchain,\n"
+                    "we are using the standard C library instead. "
+                ).format(target.name, toolchain)
+                end_warnings.append(warning)
+
             return toolchain_name, internal_tc_name, end_warnings
     else:
         if last_error:

tools/toolchains/arm.py

@@ -26,7 +26,9 @@
 from shutil import rmtree
 from distutils.version import LooseVersion
 
-from tools.toolchains.mbed_toolchain import mbedToolchain, TOOLCHAIN_PATHS
+from tools.toolchains.mbed_toolchain import (
+    mbedToolchain, TOOLCHAIN_PATHS, is_std_c_lib_supported_instead_small_c_lib
+)
 from tools.utils import mkdir, NotSupportedException, run_cmd
 from tools.resources import FileRef
 
@@ -70,7 +72,13 @@ def __init__(self, target, notify=None, macros=None,
             raise NotSupportedException(
                 "this compiler does not support the core %s" % target.core)
 
-        self.check_c_lib_supported(target, "arm")
+        toolchain = "arm"
+
+        if is_std_c_lib_supported_instead_small_c_lib(target, toolchain):
+            # Replace small C lib with std if there is no small C lib support
+            target.c_lib = "std"
+
+        self.check_c_lib_supported(target, toolchain)
 
         if (
             getattr(target, "default_toolchain", "ARM") == "uARM"
@@ -562,7 +570,13 @@ def __init__(self, target, *args, **kwargs):
                     "ARM/ARMC6 compiler support is required for ARMC6 build"
                 )
 
-        self.check_c_lib_supported(target, "arm")
+        toolchain = "arm"
+
+        if is_std_c_lib_supported_instead_small_c_lib(target, toolchain):
+            # Replace small C lib with std if there is no small C lib support
+            target.c_lib = "std"
+
+        self.check_c_lib_supported(target, toolchain)
 
         if (
             getattr(target, "default_toolchain", "ARMC6") == "uARM"

tools/toolchains/gcc.py

@@ -22,7 +22,9 @@
 from distutils.spawn import find_executable
 from distutils.version import LooseVersion
 
-from tools.toolchains.mbed_toolchain import mbedToolchain, TOOLCHAIN_PATHS
+from tools.toolchains.mbed_toolchain import (
+    mbedToolchain, TOOLCHAIN_PATHS, is_std_c_lib_supported_instead_small_c_lib
+)
 from tools.utils import run_cmd
 
 
@@ -54,7 +56,13 @@ def __init__(self, target,  notify=None, macros=None, build_profile=None,
         # Add flags for current size setting
         c_lib = "std"
         if hasattr(target, "c_lib"):
-            self.check_c_lib_supported(target, "gcc_arm")
+            toolchain = "gcc_arm"
+
+            if is_std_c_lib_supported_instead_small_c_lib(target, toolchain):
+                # Replace small C lib with std if there is no small C lib support
+                target.c_lib = "std"
+
+            self.check_c_lib_supported(target, toolchain)
             c_lib = target.c_lib
         elif hasattr(target, "default_build"):
             c_lib = target.default_build

tools/toolchains/iar.py

@@ -20,7 +20,9 @@
 from os.path import join, splitext, exists
 from distutils.version import LooseVersion
 
-from tools.toolchains.mbed_toolchain import mbedToolchain, TOOLCHAIN_PATHS
+from tools.toolchains.mbed_toolchain import (
+    mbedToolchain, TOOLCHAIN_PATHS, is_std_c_lib_supported_instead_small_c_lib
+)
 from tools.utils import run_cmd
 
 class IAR(mbedToolchain):
@@ -54,6 +56,12 @@ def __init__(self, target, notify=None, macros=None, build_profile=None,
             build_profile=build_profile
         )
 
+        toolchain = "iar"
+
+        if is_std_c_lib_supported_instead_small_c_lib(target, toolchain):
+            # Replace small C lib with std if there is no small C lib support
+            target.c_lib = "std"
+
         self.check_c_lib_supported(target, "iar")
 
         if target.is_TrustZone_secure_target:

tools/toolchains/mbed_toolchain.py

@@ -1393,3 +1393,20 @@ def report(self):
         to_ret['linker'] = {'flags': copy(self.flags['ld'])}
         to_ret.update(self.config.report)
         return to_ret
+
+def is_std_c_lib_supported_instead_small_c_lib(target, toolchain):
+    """
+    Check if the standard C lib should be used instead of the small C lib.
+    Return True if the target does not support a small C lib for a given
+    toolchain but supports the standard C library instead.
+    """
+    if (
+        hasattr(target, "c_lib")
+        and target.c_lib.lower() == "small"
+        and hasattr(target, "supported_c_libs")
+        and toolchain.lower() in target.supported_c_libs
+    ):
+        return (
+            "small" not in target.supported_c_libs[toolchain.lower()]
+            and "std" in target.supported_c_libs[toolchain.lower()]
+        )