On 05/06/18 00:12, Luc Van Oostenryck wrote:
> declare_builtin_functions() contained only the declaration of 3
> (Blackfin specific) builtin functions.
>
> Move these to declare_builtins() where all other builtin are declared.
>
> Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@xxxxxxxxx>
> ---
> builtin.c | 5 +++++
> lib.c | 18 ------------------
> 2 files changed, 5 insertions(+), 18 deletions(-)
>
> diff --git a/builtin.c b/builtin.c
> index 7f9ff3582..b409a7757 100644
> --- a/builtin.c
> +++ b/builtin.c
> @@ -459,4 +459,9 @@ void declare_builtins(void)
> declare_builtin("__sync_synchronize", &void_ctype, 0, NULL);
> declare_builtin("__sync_val_compare_and_swap", &int_ctype, 1, &ptr_ctype, NULL);
> declare_builtin("__sync_xor_and_fetch", &int_ctype, 1, &ptr_ctype, NULL);
> +
> + // Blackfin-specific stuff
> + declare_builtin("__builtin_bfin_csync", &void_ctype, 0, NULL);
> + declare_builtin("__builtin_bfin_ssync", &void_ctype, 0, NULL);
> + declare_builtin("__builtin_bfin_norm_fr1x32", &int_ctype, 0, &int_ctype, NULL);
Here, the Blackfin functions are declared unconditionally ...
> }
> diff --git a/lib.c b/lib.c
> index f9d7e094d..e4d99c5e1 100644
> --- a/lib.c
> +++ b/lib.c
> @@ -1215,23 +1215,6 @@ static void predefined_macros(void)
> predefine("__pragma__", 0, NULL);
> }
>
> -static void declare_builtin_functions(void)
> -{
> - /* Note:
> - * Most builtin functions are declared in builtin.c:declare_builtins().
> - * Some are also defined in builtin:init_builtins().
> - */
> -
> - /* Add Blackfin-specific stuff */
> - add_pre_buffer(
> - "#ifdef __bfin__\n"
> - "extern void __builtin_bfin_csync(void);\n"
> - "extern void __builtin_bfin_ssync(void);\n"
> - "extern int __builtin_bfin_norm_fr1x32(int);\n"
> - "#endif\n"
> - );
... but here they are conditional on defined(__bfin__).
ATB,
Ramsay Jones
--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
