On 3/30/21 9:30 AM, Max Filippov wrote:
> xtensa_modules variable defined in each xtensa-modules.c.inc is only
> used locally by the including file. Make it static.
> 

Reported-by: Yury Gribov <tetra2...@gmail.com>

> Signed-off-by: Max Filippov <jcmvb...@gmail.com>

Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>

> ---
>  target/xtensa/import_core.sh | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/target/xtensa/import_core.sh b/target/xtensa/import_core.sh
> index f3404039cc20..53d3c4d099bb 100755
> --- a/target/xtensa/import_core.sh
> +++ b/target/xtensa/import_core.sh
> @@ -35,6 +35,7 @@ tar -xf "$OVERLAY" -O binutils/xtensa-modules.c | \
>          -e '/^#include "ansidecl.h"/d' \
>          -e '/^Slot_[a-zA-Z0-9_]\+_decode (const xtensa_insnbuf 
> insn)/,/^}/s/^  return 0;$/  return XTENSA_UNDEFINED;/' \
>          -e 's/#include <xtensa-isa.h>/#include "xtensa-isa.h"/' \
> +        -e 's/^\(xtensa_isa_internal xtensa_modules\)/static \1/' \
>      > "$TARGET"/xtensa-modules.c.inc
>  
>  cat <<EOF > "${TARGET}.c"
> 


Reply via email to