Re: [PATCH 1/1] RISC-V: Use __bss_start for the starting point of .bss.

Kito Cheng via Newlib <[email protected]>
Newsgroups gmane.comp.lib.newlib
Message-ID <CA+yXCZCJETaEedWp0ha1=tG7YFyQ38Q+g+u9WKn_W=DcrfRNxA@mail.gmail.com>
The patch is LGTM, good catch, thanks!

On Thu, Feb 4, 2021 at 3:58 PM Yeting Kuo <[email protected]> wrote:
>
> From: Yeting Kuo <[email protected]>
>
> It's more flexible for the positions of .bss and .data.
> ---
>  libgloss/riscv/crt0.S | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libgloss/riscv/crt0.S b/libgloss/riscv/crt0.S
> index 160c07853..7a4e4e32c 100644
> --- a/libgloss/riscv/crt0.S
> +++ b/libgloss/riscv/crt0.S
> @@ -27,7 +27,7 @@ _start:
>  .option pop
>
>    # Clear the bss segment
> -  la      a0, _edata
> +  la      a0, __bss_start
>    la      a2, _end
>    sub     a2, a2, a0
>    li      a1, 0
> --
> 2.17.0
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.