[PATCH v2 2/7] Makefile: sensible default value for OPENSBI_CC_XLEN.

[Anup Patel]

On Sat, Nov 15, 2025 at 2:09 AM Benedikt Freisen <b.freisen at gmx.net> wrote:
>
> If guessing the compiler's XLEN fails, use 64 rather than garbage.
> The previous behavior could silently break e.g. OPENSBI_CC_SUPPORT_VECTOR
> when cross-compiling with a system's native clang.
>
> Signed-off-by: Benedikt Freisen <b.freisen at gmx.net>

LGTM.

Reviewed-by: Anup Patel <anup at brainfault.org>

Applied this patch to the riscv/opensbi repo.

Thanks,
Anup

> ---
>  Makefile | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/Makefile b/Makefile
> index 398eabe..afcc544 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -151,6 +151,12 @@ endif
>
>  # Guess the compiler's XLEN
>  OPENSBI_CC_XLEN := $(shell TMP=`$(CC) $(CLANG_TARGET) -dumpmachine | sed 's/riscv\([0-9][0-9]\).*/\1/'`; echo $${TMP})
> +# If guessing XLEN fails, default to 64
> +ifneq ($(OPENSBI_CC_XLEN),32)
> +  ifneq ($(OPENSBI_CC_XLEN),64)
> +    OPENSBI_CC_XLEN = 64
> +  endif
> +endif
>
>  # Guess the compiler's ABI and ISA
>  ifneq ($(CC_IS_CLANG),y)
> --
> 2.51.1
>
>
> --
> opensbi mailing list
> opensbi at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/opensbi