[PATCH] Makefile: Only enable --print-gc-section for verbose(V=1) build

[PATCH] Makefile: Only enable --print-gc-section for verbose(V=1) build

[Rahul Pathak]

Earlier this option was enabled during
debug build which only prints the linker logs
of removing the unused sections. Instead
enable this for V=1 and keep the debug build
clean.

Signed-off-by: Rahul Pathak <rpathak@ventanamicro.com>
---
 Makefile | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/Makefile b/Makefile
index 398eabe8c964..da7976d5efbb 100644
--- a/Makefile
+++ b/Makefile
@@ -444,11 +444,14 @@ DTSCPPFLAGS	=	$(CPPFLAGS) -nostdinc -nostdlib -fno-builtin -D__DTS__ -x assemble
 
 ifneq ($(DEBUG),)
 CFLAGS		+=	-O0
-ELFFLAGS	+=	-Wl,--print-gc-sections
 else
 CFLAGS		+=	-O2
 endif
 
+ifeq ($(V), 1)
+ELFFLAGS	+=	-Wl,--print-gc-sections
+endif
+
 # Setup functions for compilation
 define dynamic_flags
 -I$(shell dirname $(2)) -D__OBJNAME__=$(subst -,_,$(shell basename $(1) .o))
-- 
2.48.1


-- 
opensbi mailing list
opensbi@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/opensbi

Re: [PATCH] Makefile: Only enable --print-gc-section for verbose(V=1) build

[Anup Patel]

On Mon, Nov 10, 2025 at 10:14 PM Rahul Pathak <rpathak@ventanamicro.com> wrote:
>
> Earlier this option was enabled during
> debug build which only prints the linker logs
> of removing the unused sections. Instead
> enable this for V=1 and keep the debug build
> clean.
>
> Signed-off-by: Rahul Pathak <rpathak@ventanamicro.com>

LGTM.

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

Applied this patch to the riscv/opensbi repo.

Thanks,
Anup

> ---
>  Makefile | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index 398eabe8c964..da7976d5efbb 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -444,11 +444,14 @@ DTSCPPFLAGS       =       $(CPPFLAGS) -nostdinc -nostdlib -fno-builtin -D__DTS__ -x assemble
>
>  ifneq ($(DEBUG),)
>  CFLAGS         +=      -O0
> -ELFFLAGS       +=      -Wl,--print-gc-sections
>  else
>  CFLAGS         +=      -O2
>  endif
>
> +ifeq ($(V), 1)
> +ELFFLAGS       +=      -Wl,--print-gc-sections
> +endif
> +
>  # Setup functions for compilation
>  define dynamic_flags
>  -I$(shell dirname $(2)) -D__OBJNAME__=$(subst -,_,$(shell basename $(1) .o))
> --
> 2.48.1
>
>
> --
> opensbi mailing list
> opensbi@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/opensbi

-- 
opensbi mailing list
opensbi@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/opensbi