Re: [PATCH 1/1] gensymlist: fix clang build with exporting of __builtin_trap

[Vladimir 'phcoder' Serbinenko]

On Tue, Jun 14, 2022 at 3:37 PM Darren Kenny <darren.kenny@oracle.com> wrote:
>
> clang expands the abort function to __builtin_trap, but that cannot be
> exported.
>
> The script that generates the symlist.c file should also exclude any
> symbols that start with __builtin_.
This patch will break many platforms including PPC which rely on
functions like __builtin_bswap*
>
> Signed-off-by: Darren Kenny <darren.kenny@oracle.com>
> ---
>  grub-core/gensymlist.sh | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/grub-core/gensymlist.sh b/grub-core/gensymlist.sh
> index 5074ef6aad58..a2e5b85d0a71 100644
> --- a/grub-core/gensymlist.sh
> +++ b/grub-core/gensymlist.sh
> @@ -58,6 +58,7 @@ EOF
>
>  (while read LINE; do echo $LINE; done) \
>    | grep -v '^#' \
> +  | grep -v 'EXPORT_FUNC *(__builtin_[a-zA-Z0-9_]*)' \
>    | sed -n \
>          -e '/EXPORT_FUNC *([a-zA-Z0-9_]*)/{s/.*EXPORT_FUNC 
> *(\([a-zA-Z0-9_]*\)).*/      {"\1", \1, 1},/;p;}' \
>          -e '/EXPORT_VAR *([a-zA-Z0-9_]*)/{s/.*EXPORT_VAR 
> *(\([a-zA-Z0-9_]*\)).*/      {"\1", (void *) \&\1, 0},/;p;}' \
> --
> 2.31.1
>
>
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> https://lists.gnu.org/mailman/listinfo/grub-devel



-- 
Regards
Vladimir 'phcoder' Serbinenko