This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Patches to fix GCCâs C++ exception handling on NetBSD/VAX
- From: Jake Hamby <jehamby420 at me dot com>
- To: Christos Zoulas <christos at astron dot com>
- Cc: port-vax at netbsd dot org, gcc-patches at gcc dot gnu dot org
- Date: Sat, 26 Mar 2016 07:38:54 -0700
- Subject: Re: Patches to fix GCCâs C++ exception handling on NetBSD/VAX
- Authentication-results: sourceware.org; auth=none
- References: <F48D0C6B-A6DB-410B-BC97-C30D4E8B4612 at me dot com> <ncu3pc$gkd$1 at ger dot gmane dot org> <DB0276A6-0A26-44F7-8FDC-D086EA40C709 at me dot com>
Unfortunately, my previous patch that included a change to gcc/config/vax/vax.h that increased FIRST_PSEUDO_REGISTER from 16 to 17 breaks the C++ exception handling that Iâd worked so hard to get right with the rest of the patch. I believe I need to define DWARF_FRAME_REGISTERS to 16 in the same file to fix the size of the array that libgcc/unwind-dw2.c creates. The i386 backend and several others also define it their .h file for the same reason (compatibility with hardcoded frame offsets).
Hereâs the first part of the patch to vax.h that increases FIRST_PSEUDO_REGISTER and also adds a definition of DWARF_FRAME_REGISTERS as 16, with suitable comment. Iâm testing it now. I know that C++ exceptions were working before I increased FIRST_PSEUDO_REGISTER to 17.
Regards,
Jake
Index: external/gpl3/gcc.old/dist/gcc/config/vax/vax.h
===================================================================
RCS file: /cvsroot/src/external/gpl3/gcc.old/dist/gcc/config/vax/vax.h,v
retrieving revision 1.3
diff -u -r1.3 vax.h
--- external/gpl3/gcc.old/dist/gcc/config/vax/vax.h 23 Sep 2015 03:39:18 -0000 1.3
+++ external/gpl3/gcc.old/dist/gcc/config/vax/vax.h 26 Mar 2016 14:34:29 -0000
@@ -119,13 +119,17 @@
The hardware registers are assigned numbers for the compiler
from 0 to just below FIRST_PSEUDO_REGISTER.
All registers that the compiler knows about must be given numbers,
- even those that are not normally considered general registers. */
-#define FIRST_PSEUDO_REGISTER 16
+ even those that are not normally considered general registers.
+ This includes PSW, which the VAX backend did not originally include. */
+#define FIRST_PSEUDO_REGISTER 17
+
+/* For compatibility, DWARF_FRAME_REGISTERS must still be 16. */
+#define DWARF_FRAME_REGISTERS 16
/* 1 for registers that have pervasive standard uses
and are not available for the register allocator.
- On the VAX, these are the AP, FP, SP and PC. */
-#define FIXED_REGISTERS {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1}
+ On the VAX, these are the AP, FP, SP, PC, and PSW. */
+#define FIXED_REGISTERS {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1}
/* 1 for registers not available across function calls.
These must include the FIXED_REGISTERS and also any