[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gcc-patches
Subject:    [PATCH] RISC-V: Add target machine headers as a dependency for riscv-sr.o
From:       "Maciej W. Rozycki" <macro () embecosm ! com>
Date:       2022-01-31 21:50:12
Message-ID: alpine.DEB.2.20.2201312132050.11348 () tpp ! orcam ! me ! uk
[Download RAW message or body]

Make riscv-sr.o depend on target machine headers, removing spurious test
failures:

FAIL: gcc.target/riscv/save-restore-3.c scan-assembler-not call[ \t]*t0,__riscv_save_0
FAIL: gcc.target/riscv/save-restore-3.c scan-assembler-not tail[ \t]*__riscv_restore_0
FAIL: gcc.target/riscv/save-restore-3.c scan-assembler tail[ \t]*foo
FAIL: gcc.target/riscv/save-restore-6.c scan-assembler-not call[ \t]*t0,__riscv_save_0
FAIL: gcc.target/riscv/save-restore-6.c scan-assembler-not tail[ \t]*__riscv_restore_0
FAIL: gcc.target/riscv/save-restore-6.c scan-assembler tail[ \t]*other_func

if the definitions of UNSPECs are locally changed and GCC rebuilt from a 
dirty tree.

	gcc/
	* config/riscv/t-riscv (riscv-sr.o): Add $(TM_H) dependency.
---
Hi,

 Noticed while fiddling with `fmin'/`fmax' further.  As not a code change 
OK to apply despite stage 4?  Might be worth backporting too.

  Maciej
---
 gcc/config/riscv/t-riscv |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

gcc-riscv-sr-dep.diff
Index: gcc/gcc/config/riscv/t-riscv
===================================================================
--- gcc.orig/gcc/config/riscv/t-riscv
+++ gcc/gcc/config/riscv/t-riscv
@@ -6,7 +6,7 @@ riscv-builtins.o: $(srcdir)/config/riscv
 		$(srcdir)/config/riscv/riscv-builtins.cc
 
 riscv-sr.o: $(srcdir)/config/riscv/riscv-sr.cc $(CONFIG_H) \
-  $(SYSTEM_H)
+  $(SYSTEM_H) $(TM_H)
 	$(COMPILER) -c $(ALL_COMPILERFLAGS) $(ALL_CPPFLAGS) $(INCLUDES) \
 		$(srcdir)/config/riscv/riscv-sr.cc
 
[prev in list] [next in list] [prev in thread] [next in thread]