[PATCH v4 1/3] RISC-V: Add include guard for dl-tls.h.

Fangrui Song maskray@google.com
Sat Jan 27 01:14:26 GMT 2024


On Thu, Sep 14, 2023 at 1:41 AM Tatsuyuki Ishi <ishitatsuyuki@gmail.com> wrote:
>
> ---
>  sysdeps/riscv/dl-tls.h | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/sysdeps/riscv/dl-tls.h b/sysdeps/riscv/dl-tls.h
> index 67c8ae639c..f11f335edf 100644
> --- a/sysdeps/riscv/dl-tls.h
> +++ b/sysdeps/riscv/dl-tls.h
> @@ -16,6 +16,8 @@
>     License along with the GNU C Library.  If not, see
>     <https://www.gnu.org/licenses/>.  */
>
> +#ifndef _DL_TLS_H
> +# define _DL_TLS_H 1
>
>  /* Type used for the representation of TLS information in the GOT.  */
>  typedef struct
> @@ -46,3 +48,5 @@ extern void *__tls_get_addr (tls_index *ti);
>
>  /* Value used for dtv entries for which the allocation is delayed.  */
>  #define TLS_DTV_UNALLOCATED    ((void *) -1l)
> +
> +#endif /* _DL_TLS_H */
> --
> 2.42.0
>

This commit message should state why the header is needed.
Multiple sysdeps/*/dl-tls.h do not have a guard and they are good.


-- 
宋方睿


More information about the Libc-alpha mailing list