diff mbox

[9/9] c-family/c-cppbuiltin.c fix

Message ID b6c3e349-4b48-caa4-c730-f7daf1bc8332@codesourcery.com
State New
Headers show

Commit Message

Andrew Jenner April 1, 2017, 4:54 p.m. UTC
I needed to apply the attached patch for ia16, so that 
__LIBGCC_JCR_SECTION_NAME__ does not get defined unless 
TARGET_USE_JCR_SECTION is.

2017-04-01  Andrew Jenner  <andrew@codesourcery.com>

     * c-family/g-cppbuiltin.c (c_cpp_builtins): guard 
__LIBGCC_JCR_SECTION_NAME__ definition with TARGET_USE_JCR_SECTION.

Comments

Andrew Pinski April 1, 2017, 4:56 p.m. UTC | #1
On Sat, Apr 1, 2017 at 9:54 AM, Andrew Jenner <andrew@codesourcery.com> wrote:
> I needed to apply the attached patch for ia16, so that
> __LIBGCC_JCR_SECTION_NAME__ does not get defined unless
> TARGET_USE_JCR_SECTION is.
>
> 2017-04-01  Andrew Jenner  <andrew@codesourcery.com>
>
>     * c-family/g-cppbuiltin.c (c_cpp_builtins): guard
> __LIBGCC_JCR_SECTION_NAME__ definition with TARGET_USE_JCR_SECTION.

Actually TARGET_USE_JCR_SECTION support should be removed (no April
fools joke).  Since the Java front-end is no gone.

Thanks,
Andrew
diff mbox

Patch

Index: gcc/c-family/c-cppbuiltin.c
===================================================================
--- gcc/c-family/c-cppbuiltin.c	(revision 475331)
+++ gcc/c-family/c-cppbuiltin.c	(revision 475455)
@@ -1119,10 +1119,12 @@  c_cpp_builtins (cpp_reader *pfile)
       builtin_define_with_value ("__LIBGCC_EH_FRAME_SECTION_NAME__",
 				 EH_FRAME_SECTION_NAME, 1);
 #endif
+#if TARGET_USE_JCR_SECTION
 #ifdef JCR_SECTION_NAME
       builtin_define_with_value ("__LIBGCC_JCR_SECTION_NAME__",
 				 JCR_SECTION_NAME, 1);
 #endif
+#endif
 #ifdef CTORS_SECTION_ASM_OP
       builtin_define_with_value ("__LIBGCC_CTORS_SECTION_ASM_OP__",
 				 CTORS_SECTION_ASM_OP, 1);