[PATCH] RISC-V: Fix -Wint-conversion warning

Kito Cheng kito.cheng@sifive.com
Mon Dec 4 07:41:28 GMT 2023


Upstream GCC has change this warning into error by default, so...we need
to explicitly convert the type from pointer from/to integer, generally
it's unsafe, but we know what we are doing here.

However it's not safe for ilp32 on RV64, but we didn't support that yet, so I
think this fix is good enough now :)
---
 libgloss/riscv/internal_syscall.h | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/libgloss/riscv/internal_syscall.h b/libgloss/riscv/internal_syscall.h
index 1f1f42f5d..080c8c847 100644
--- a/libgloss/riscv/internal_syscall.h
+++ b/libgloss/riscv/internal_syscall.h
@@ -63,7 +63,7 @@ __internal_syscall(long n, int argc, long _a0, long _a1, long _a2, long _a3, lon
 }
 
 static inline long
-syscall_errno(long n, int argc, long _a0, long _a1, long _a2, long _a3, long _a4, long _a5)
+_syscall_errno(long n, int argc, long _a0, long _a1, long _a2, long _a3, long _a4, long _a5)
 {
   long a0 = __internal_syscall (n, argc, _a0, _a1, _a2, _a3, _a4, _a5);
 
@@ -73,4 +73,8 @@ syscall_errno(long n, int argc, long _a0, long _a1, long _a2, long _a3, long _a4
     return a0;
 }
 
+#define syscall_errno(N, ARGC, A0, A1, A2, A3, A4, A5) \
+  _syscall_errno(N, ARGC, (long)A0, (long)A1, (long)A2, \
+	         (long)A3, (long)A4, (long)A5)
+
 #endif
-- 
2.40.1



More information about the Newlib mailing list