public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [PATCH] config/rs6000/t-float128: Don't encode full build paths into headers
@ 2022-08-17 12:10 Richard Purdie
  2022-11-02 10:45 ` Richard Purdie
  0 siblings, 1 reply; 2+ messages in thread
From: Richard Purdie @ 2022-08-17 12:10 UTC (permalink / raw)
  To: gcc-patches

Avoid encoding full build paths into headers, just use the basename of the file.
This aids build reproducibility where the build paths vary and source is saved
for debugging purposes.

libgcc/ChangeLog:

    * config/rs6000/t-float128: Don't encode full build paths into headers

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 libgcc/config/rs6000/t-float128 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libgcc/config/rs6000/t-float128 b/libgcc/config/rs6000/t-float128
index b09b5664af0..513e63748f1 100644
--- a/libgcc/config/rs6000/t-float128
+++ b/libgcc/config/rs6000/t-float128
@@ -103,7 +103,7 @@ $(ibm128_dec_objs)	: INTERNAL_CFLAGS += $(IBM128_CFLAGS_DECIMAL)
 $(fp128_softfp_src) : $(srcdir)/soft-fp/$(subst -sw,,$(subst kf,tf,$@)) $(fp128_dep)
 	@src="$(srcdir)/soft-fp/$(subst -sw,,$(subst kf,tf,$@))"; \
 	echo "Create $@"; \
-	(echo "/* file created from $$src */"; \
+	(echo "/* file created from `basename $$src` */"; \
 	 echo; \
 	 sed -f $(fp128_sed) < $$src) > $@
 
-- 
2.34.1


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH] config/rs6000/t-float128: Don't encode full build paths into headers
  2022-08-17 12:10 [PATCH] config/rs6000/t-float128: Don't encode full build paths into headers Richard Purdie
@ 2022-11-02 10:45 ` Richard Purdie
  0 siblings, 0 replies; 2+ messages in thread
From: Richard Purdie @ 2022-11-02 10:45 UTC (permalink / raw)
  To: gcc-patches

On Wed, 2022-08-17 at 13:10 +0100, Richard Purdie via Gcc-patches
wrote:
> Avoid encoding full build paths into headers, just use the basename of the file.
> This aids build reproducibility where the build paths vary and source is saved
> for debugging purposes.
> 
> libgcc/ChangeLog:
> 
>     * config/rs6000/t-float128: Don't encode full build paths into headers
> 

I think this patch is at risk of being lost. It is a simple change
which aids reproducibility so I'm hoping someone might be able to help
with review/merging?

Thanks!

Richard


> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> ---
>  libgcc/config/rs6000/t-float128 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libgcc/config/rs6000/t-float128 b/libgcc/config/rs6000/t-float128
> index b09b5664af0..513e63748f1 100644
> --- a/libgcc/config/rs6000/t-float128
> +++ b/libgcc/config/rs6000/t-float128
> @@ -103,7 +103,7 @@ $(ibm128_dec_objs)	: INTERNAL_CFLAGS += $(IBM128_CFLAGS_DECIMAL)
>  $(fp128_softfp_src) : $(srcdir)/soft-fp/$(subst -sw,,$(subst kf,tf,$@)) $(fp128_dep)
>  	@src="$(srcdir)/soft-fp/$(subst -sw,,$(subst kf,tf,$@))"; \
>  	echo "Create $@"; \
> -	(echo "/* file created from $$src */"; \
> +	(echo "/* file created from `basename $$src` */"; \
>  	 echo; \
>  	 sed -f $(fp128_sed) < $$src) > $@
>  




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-11-02 10:45 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-17 12:10 [PATCH] config/rs6000/t-float128: Don't encode full build paths into headers Richard Purdie
2022-11-02 10:45 ` Richard Purdie

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).