Hi!
OK to push the attached "Modula-2: Support '-isysroot [...]'"?
This greatly improves test results for the cross configurations I've
tested, but I don't know if any real handling needs to be implemented, or
this should be done differently altogether?
Grüße
Thomas
-----------------
Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634
München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas
Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht
München, HRB 106955
>From 0bd30fd25138497df5320e5f63fd04e1b5756cc5 Mon Sep 17 00:00:00 2001
From: Thomas Schwinge <tho...@codesourcery.com>
Date: Tue, 5 Dec 2023 09:54:54 +0100
Subject: [PATCH] Modula-2: Support '-isysroot [...]'
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
In GCC cross configurations (tested '--target=amdgcn-amdhsa' and
'--target=nvptx-none') with a sysroot configured, the 'gm2' driver invocations
are passed '--sysroot=[...]', which is translated into '-isysroot [...]' for
the 'cc1gm2' compiler invocation. The latter, however gets complained about:
cc1gm2: warning: command-line option ‘-isysroot [...]’ is valid for C/C++/D/Fortran/ObjC/ObjC++ but not for Modula-2
..., and therefore a ton of FAILs.
Reproducer (also for non-cross, native configurations):
$ build-gcc/gcc/gm2 -Bbuild-gcc/gcc -v --sysroot=/tmp -x modula-2 /dev/null
[...]
build-gcc/gcc/cc1gm2 [...] -isysroot [...]/tmp [...]
cc1gm2: warning: command-line option ‘-isysroot /tmp’ is valid for C/C++/D/Fortran/ObjC/ObjC++ but not for Modula-2
[...]
gcc/m2/
* lang.opt (-isysroot): New.
---
gcc/m2/lang.opt | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/gcc/m2/lang.opt b/gcc/m2/lang.opt
index 24f3c6594b9..a60c03e69d4 100644
--- a/gcc/m2/lang.opt
+++ b/gcc/m2/lang.opt
@@ -405,6 +405,10 @@ iquote
Modula-2
; Documented in c.opt
+isysroot
+Modula-2
+; Documented in c.opt
+
isystem
Modula-2
; Documented in c.opt
--
2.34.1
