* Re: [PATCH 7/7] Plug ipa-prop escape analysis into gimple_call_arg_flags
@ 2014-05-26 9:00 Dominique Dhumieres
2014-05-26 9:16 ` Andrew Pinski
0 siblings, 1 reply; 10+ messages in thread
From: Dominique Dhumieres @ 2014-05-26 9:00 UTC (permalink / raw)
To: gcc-patches; +Cc: richard.guenther, hubicka
r210901 breaks bootstrap on targets not supporting strnlen, e.g., darwin10.
../../_clean/gcc/lto-cgraph.c:976:68: error: 'strnlen' was not declared in this scope
libgfortran/configure defines HAVE_STRNLEN on targets supporting it.
The same revision also breaks the test g++.dg/tls/diag-1.C
/opt/gcc/work/gcc/testsuite/g++.dg/tls/diag-1.C:31:1: internal compiler error: in symtab_get_node, at cgraph.h:1021
TIA,
Dominique
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 7/7] Plug ipa-prop escape analysis into gimple_call_arg_flags
2014-05-26 9:00 [PATCH 7/7] Plug ipa-prop escape analysis into gimple_call_arg_flags Dominique Dhumieres
@ 2014-05-26 9:16 ` Andrew Pinski
2014-05-26 9:29 ` Christophe Lyon
2014-05-28 7:26 ` Thomas Schwinge
0 siblings, 2 replies; 10+ messages in thread
From: Andrew Pinski @ 2014-05-26 9:16 UTC (permalink / raw)
To: Dominique Dhumieres; +Cc: GCC Patches, Richard Guenther, Jan Hubicka
On Mon, May 26, 2014 at 1:59 AM, Dominique Dhumieres <dominiq@lps.ens.fr> wrote:
> r210901 breaks bootstrap on targets not supporting strnlen, e.g., darwin10.
>
> ../../_clean/gcc/lto-cgraph.c:976:68: error: 'strnlen' was not declared in this scope
strnlen should be declared in include/libiberty.h if there is no
declaration as libiberty support is already there. That should be a
simple fix.
Thanks,
Andrew Pinski
>
> libgfortran/configure defines HAVE_STRNLEN on targets supporting it.
>
> The same revision also breaks the test g++.dg/tls/diag-1.C
>
> /opt/gcc/work/gcc/testsuite/g++.dg/tls/diag-1.C:31:1: internal compiler error: in symtab_get_node, at cgraph.h:1021
>
> TIA,
>
> Dominique
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 7/7] Plug ipa-prop escape analysis into gimple_call_arg_flags
2014-05-26 9:16 ` Andrew Pinski
@ 2014-05-26 9:29 ` Christophe Lyon
2014-05-26 16:32 ` Jan Hubicka
2014-05-28 7:26 ` Thomas Schwinge
1 sibling, 1 reply; 10+ messages in thread
From: Christophe Lyon @ 2014-05-26 9:29 UTC (permalink / raw)
To: Andrew Pinski
Cc: Dominique Dhumieres, GCC Patches, Richard Guenther, Jan Hubicka
On my side, I can see that r210901 breaks AArch64 compiler build:
gcc/config/aarch64/aarch64.c: In function ‘void
aarch64_elf_asm_named_section(const char*, unsigned int, tree_node*)’:
gcc/config/aarch64/aarch64.c:8136: error: ‘decl_comdat_group’ was not
declared in this scope
Christophe.
On 26 May 2014 11:16, Andrew Pinski <pinskia@gmail.com> wrote:
> On Mon, May 26, 2014 at 1:59 AM, Dominique Dhumieres <dominiq@lps.ens.fr> wrote:
>> r210901 breaks bootstrap on targets not supporting strnlen, e.g., darwin10.
>>
>> ../../_clean/gcc/lto-cgraph.c:976:68: error: 'strnlen' was not declared in this scope
>
> strnlen should be declared in include/libiberty.h if there is no
> declaration as libiberty support is already there. That should be a
> simple fix.
>
> Thanks,
> Andrew Pinski
>
>>
>> libgfortran/configure defines HAVE_STRNLEN on targets supporting it.
>>
>> The same revision also breaks the test g++.dg/tls/diag-1.C
>>
>> /opt/gcc/work/gcc/testsuite/g++.dg/tls/diag-1.C:31:1: internal compiler error: in symtab_get_node, at cgraph.h:1021
>>
>> TIA,
>>
>> Dominique
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 7/7] Plug ipa-prop escape analysis into gimple_call_arg_flags
2014-05-26 9:29 ` Christophe Lyon
@ 2014-05-26 16:32 ` Jan Hubicka
0 siblings, 0 replies; 10+ messages in thread
From: Jan Hubicka @ 2014-05-26 16:32 UTC (permalink / raw)
To: Christophe Lyon
Cc: Andrew Pinski, Dominique Dhumieres, GCC Patches,
Richard Guenther, Jan Hubicka
> On my side, I can see that r210901 breaks AArch64 compiler build:
> gcc/config/aarch64/aarch64.c: In function âvoid
> aarch64_elf_asm_named_section(const char*, unsigned int, tree_node*)â:
> gcc/config/aarch64/aarch64.c:8136: error: âdecl_comdat_groupâ was not
> declared in this scope
This sould be fixed by adding #include "cgraph.h" into aarch64.c
I will look into the strnlen issue.
Honza
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 7/7] Plug ipa-prop escape analysis into gimple_call_arg_flags
2014-05-26 9:16 ` Andrew Pinski
2014-05-26 9:29 ` Christophe Lyon
@ 2014-05-28 7:26 ` Thomas Schwinge
2014-05-28 21:55 ` Jan Hubicka
1 sibling, 1 reply; 10+ messages in thread
From: Thomas Schwinge @ 2014-05-28 7:26 UTC (permalink / raw)
To: Andrew Pinski, Dominique Dhumieres, Jan Hubicka
Cc: GCC Patches, Richard Guenther
[-- Attachment #1: Type: text/plain, Size: 1876 bytes --]
Hi!
On Mon, 26 May 2014 02:16:35 -0700, Andrew Pinski <pinskia@gmail.com> wrote:
> On Mon, May 26, 2014 at 1:59 AM, Dominique Dhumieres <dominiq@lps.ens.fr> wrote:
> > r210901 breaks bootstrap on targets not supporting strnlen, e.g., darwin10.
> >
> > ../../_clean/gcc/lto-cgraph.c:976:68: error: 'strnlen' was not declared in this scope
I'm seeing the same on MinGW, which also doesn't have strnlen (which is a
GNU extension).
> strnlen should be declared in include/libiberty.h if there is no
> declaration as libiberty support is already there. That should be a
> simple fix.
Like this?
--- gcc/config.in
+++ gcc/config.in
[Regenerate.]
--- gcc/configure
+++ gcc/configure
[Regenerate.]
--- gcc/configure.ac
+++ gcc/configure.ac
@@ -1136,7 +1136,7 @@ CFLAGS="$CFLAGS -I${srcdir} -I${srcdir}/../include $GMPINC"
saved_CXXFLAGS="$CXXFLAGS"
CXXFLAGS="$CXXFLAGS -I${srcdir} -I${srcdir}/../include $GMPINC"
gcc_AC_CHECK_DECLS(getenv atol asprintf sbrk abort atof getcwd getwd \
- strsignal strstr stpcpy strverscmp \
+ stpcpy strnlen strsignal strstr strverscmp \
errno snprintf vsnprintf vasprintf malloc realloc calloc \
free basename getopt clock getpagesize ffs gcc_UNLOCKED_FUNCS, , ,[
#include "ansidecl.h"
diff --git include/libiberty.h include/libiberty.h
index 7fd0703..56b8b43 100644
--- include/libiberty.h
+++ include/libiberty.h
@@ -636,6 +636,10 @@ extern int snprintf (char *, size_t, const char *, ...) ATTRIBUTE_PRINTF_3;
extern int vsnprintf (char *, size_t, const char *, va_list) ATTRIBUTE_PRINTF(3,0);
#endif
+#if defined (HAVE_DECL_STRNLEN) && !HAVE_DECL_STRNLEN
+extern size_t strnlen (const char *, size_t);
+#endif
+
#if defined(HAVE_DECL_STRVERSCMP) && !HAVE_DECL_STRVERSCMP
/* Compare version strings. */
extern int strverscmp (const char *, const char *);
Grüße,
Thomas
[-- Attachment #2: Type: application/pgp-signature, Size: 472 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 7/7] Plug ipa-prop escape analysis into gimple_call_arg_flags
2014-05-28 7:26 ` Thomas Schwinge
@ 2014-05-28 21:55 ` Jan Hubicka
2014-06-03 9:56 ` Thomas Schwinge
0 siblings, 1 reply; 10+ messages in thread
From: Jan Hubicka @ 2014-05-28 21:55 UTC (permalink / raw)
To: Thomas Schwinge
Cc: Andrew Pinski, Dominique Dhumieres, Jan Hubicka, GCC Patches,
Richard Guenther
> Hi!
>
> On Mon, 26 May 2014 02:16:35 -0700, Andrew Pinski <pinskia@gmail.com> wrote:
> > On Mon, May 26, 2014 at 1:59 AM, Dominique Dhumieres <dominiq@lps.ens.fr> wrote:
> > > r210901 breaks bootstrap on targets not supporting strnlen, e.g., darwin10.
> > >
> > > ../../_clean/gcc/lto-cgraph.c:976:68: error: 'strnlen' was not declared in this scope
>
> I'm seeing the same on MinGW, which also doesn't have strnlen (which is a
> GNU extension).
>
> > strnlen should be declared in include/libiberty.h if there is no
> > declaration as libiberty support is already there. That should be a
> > simple fix.
>
> Like this?
This looks good to me (thoguh strnlen is posix). I can not approve the patch
but I would preffer it over just hand implementing strnlen there (that is easy,
too)
Honza
>
> --- gcc/config.in
> +++ gcc/config.in
> [Regenerate.]
> --- gcc/configure
> +++ gcc/configure
> [Regenerate.]
> --- gcc/configure.ac
> +++ gcc/configure.ac
> @@ -1136,7 +1136,7 @@ CFLAGS="$CFLAGS -I${srcdir} -I${srcdir}/../include $GMPINC"
> saved_CXXFLAGS="$CXXFLAGS"
> CXXFLAGS="$CXXFLAGS -I${srcdir} -I${srcdir}/../include $GMPINC"
> gcc_AC_CHECK_DECLS(getenv atol asprintf sbrk abort atof getcwd getwd \
> - strsignal strstr stpcpy strverscmp \
> + stpcpy strnlen strsignal strstr strverscmp \
> errno snprintf vsnprintf vasprintf malloc realloc calloc \
> free basename getopt clock getpagesize ffs gcc_UNLOCKED_FUNCS, , ,[
> #include "ansidecl.h"
> diff --git include/libiberty.h include/libiberty.h
> index 7fd0703..56b8b43 100644
> --- include/libiberty.h
> +++ include/libiberty.h
> @@ -636,6 +636,10 @@ extern int snprintf (char *, size_t, const char *, ...) ATTRIBUTE_PRINTF_3;
> extern int vsnprintf (char *, size_t, const char *, va_list) ATTRIBUTE_PRINTF(3,0);
> #endif
>
> +#if defined (HAVE_DECL_STRNLEN) && !HAVE_DECL_STRNLEN
> +extern size_t strnlen (const char *, size_t);
> +#endif
> +
> #if defined(HAVE_DECL_STRVERSCMP) && !HAVE_DECL_STRVERSCMP
> /* Compare version strings. */
> extern int strverscmp (const char *, const char *);
>
>
> GrĂźĂe,
> Thomas
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 7/7] Plug ipa-prop escape analysis into gimple_call_arg_flags
2014-05-28 21:55 ` Jan Hubicka
@ 2014-06-03 9:56 ` Thomas Schwinge
2014-06-10 6:48 ` Commit policy? " Thomas Schwinge
0 siblings, 1 reply; 10+ messages in thread
From: Thomas Schwinge @ 2014-06-03 9:56 UTC (permalink / raw)
To: Dominique Dhumieres, Jan Hubicka, GCC Patches
Cc: Andrew Pinski, Richard Guenther, Jan Hubicka
[-- Attachment #1: Type: text/plain, Size: 2422 bytes --]
Hi!
Ping -- OK to commit to trunk?
On Wed, 28 May 2014 23:55:31 +0200, Jan Hubicka <hubicka@ucw.cz> wrote:
> > On Mon, 26 May 2014 02:16:35 -0700, Andrew Pinski <pinskia@gmail.com> wrote:
> > > On Mon, May 26, 2014 at 1:59 AM, Dominique Dhumieres <dominiq@lps.ens.fr> wrote:
> > > > r210901 breaks bootstrap on targets not supporting strnlen, e.g., darwin10.
> > > >
> > > > ../../_clean/gcc/lto-cgraph.c:976:68: error: 'strnlen' was not declared in this scope
> >
> > I'm seeing the same on MinGW, which also doesn't have strnlen (which is a
> > GNU extension).
> >
> > > strnlen should be declared in include/libiberty.h if there is no
> > > declaration as libiberty support is already there. That should be a
> > > simple fix.
> >
> > Like this?
>
> This looks good to me (thoguh strnlen is posix). I can not approve the patch
> but I would preffer it over just hand implementing strnlen there (that is easy,
> too)
Patch is also considered good by testers in
<https://gcc.gnu.org/PR61334>.
> > --- gcc/config.in
> > +++ gcc/config.in
> > [Regenerate.]
> > --- gcc/configure
> > +++ gcc/configure
> > [Regenerate.]
> > --- gcc/configure.ac
> > +++ gcc/configure.ac
> > @@ -1136,7 +1136,7 @@ CFLAGS="$CFLAGS -I${srcdir} -I${srcdir}/../include $GMPINC"
> > saved_CXXFLAGS="$CXXFLAGS"
> > CXXFLAGS="$CXXFLAGS -I${srcdir} -I${srcdir}/../include $GMPINC"
> > gcc_AC_CHECK_DECLS(getenv atol asprintf sbrk abort atof getcwd getwd \
> > - strsignal strstr stpcpy strverscmp \
> > + stpcpy strnlen strsignal strstr strverscmp \
> > errno snprintf vsnprintf vasprintf malloc realloc calloc \
> > free basename getopt clock getpagesize ffs gcc_UNLOCKED_FUNCS, , ,[
> > #include "ansidecl.h"
> > diff --git include/libiberty.h include/libiberty.h
> > index 7fd0703..56b8b43 100644
> > --- include/libiberty.h
> > +++ include/libiberty.h
> > @@ -636,6 +636,10 @@ extern int snprintf (char *, size_t, const char *, ...) ATTRIBUTE_PRINTF_3;
> > extern int vsnprintf (char *, size_t, const char *, va_list) ATTRIBUTE_PRINTF(3,0);
> > #endif
> >
> > +#if defined (HAVE_DECL_STRNLEN) && !HAVE_DECL_STRNLEN
> > +extern size_t strnlen (const char *, size_t);
> > +#endif
> > +
> > #if defined(HAVE_DECL_STRVERSCMP) && !HAVE_DECL_STRVERSCMP
> > /* Compare version strings. */
> > extern int strverscmp (const char *, const char *);
Grüße,
Thomas
[-- Attachment #2: Type: application/pgp-signature, Size: 472 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Commit policy? Re: [PATCH 7/7] Plug ipa-prop escape analysis into gimple_call_arg_flags
2014-06-03 9:56 ` Thomas Schwinge
@ 2014-06-10 6:48 ` Thomas Schwinge
2014-06-10 6:54 ` Andrew Pinski
2014-06-10 9:07 ` Richard Biener
0 siblings, 2 replies; 10+ messages in thread
From: Thomas Schwinge @ 2014-06-10 6:48 UTC (permalink / raw)
To: GCC Patches
Cc: Dominique Dhumieres, Jan Hubicka, Andrew Pinski,
Richard Guenther, Jan Hubicka, dave.anglin, ro, iains
[-- Attachment #1: Type: text/plain, Size: 2941 bytes --]
Hi!
On Tue, 3 Jun 2014 11:55:44 +0200, I wrote:
> Ping -- OK to commit to trunk?
Even though several of those who I'd consider regular GCC developers do
agree with this patch (see also <https://gcc.gnu.org/PR61334>), that is
not enough "consensus" to consider this patch approved, right? Wouldn't
it be a good idea for GCC to move to a more "liberal" policy in this
regard? (That seems to work nicely for glibc.)
Ping.
> On Wed, 28 May 2014 23:55:31 +0200, Jan Hubicka <hubicka@ucw.cz> wrote:
> > > On Mon, 26 May 2014 02:16:35 -0700, Andrew Pinski <pinskia@gmail.com> wrote:
> > > > On Mon, May 26, 2014 at 1:59 AM, Dominique Dhumieres <dominiq@lps.ens.fr> wrote:
> > > > > r210901 breaks bootstrap on targets not supporting strnlen, e.g., darwin10.
> > > > >
> > > > > ../../_clean/gcc/lto-cgraph.c:976:68: error: 'strnlen' was not declared in this scope
> > >
> > > I'm seeing the same on MinGW, which also doesn't have strnlen (which is a
> > > GNU extension).
> > >
> > > > strnlen should be declared in include/libiberty.h if there is no
> > > > declaration as libiberty support is already there. That should be a
> > > > simple fix.
> > >
> > > Like this?
> >
> > This looks good to me (thoguh strnlen is posix). I can not approve the patch
> > but I would preffer it over just hand implementing strnlen there (that is easy,
> > too)
>
> Patch is also considered good by testers in
> <https://gcc.gnu.org/PR61334>.
>
> > > --- gcc/config.in
> > > +++ gcc/config.in
> > > [Regenerate.]
> > > --- gcc/configure
> > > +++ gcc/configure
> > > [Regenerate.]
> > > --- gcc/configure.ac
> > > +++ gcc/configure.ac
> > > @@ -1136,7 +1136,7 @@ CFLAGS="$CFLAGS -I${srcdir} -I${srcdir}/../include $GMPINC"
> > > saved_CXXFLAGS="$CXXFLAGS"
> > > CXXFLAGS="$CXXFLAGS -I${srcdir} -I${srcdir}/../include $GMPINC"
> > > gcc_AC_CHECK_DECLS(getenv atol asprintf sbrk abort atof getcwd getwd \
> > > - strsignal strstr stpcpy strverscmp \
> > > + stpcpy strnlen strsignal strstr strverscmp \
> > > errno snprintf vsnprintf vasprintf malloc realloc calloc \
> > > free basename getopt clock getpagesize ffs gcc_UNLOCKED_FUNCS, , ,[
> > > #include "ansidecl.h"
> > > diff --git include/libiberty.h include/libiberty.h
> > > index 7fd0703..56b8b43 100644
> > > --- include/libiberty.h
> > > +++ include/libiberty.h
> > > @@ -636,6 +636,10 @@ extern int snprintf (char *, size_t, const char *, ...) ATTRIBUTE_PRINTF_3;
> > > extern int vsnprintf (char *, size_t, const char *, va_list) ATTRIBUTE_PRINTF(3,0);
> > > #endif
> > >
> > > +#if defined (HAVE_DECL_STRNLEN) && !HAVE_DECL_STRNLEN
> > > +extern size_t strnlen (const char *, size_t);
> > > +#endif
> > > +
> > > #if defined(HAVE_DECL_STRVERSCMP) && !HAVE_DECL_STRVERSCMP
> > > /* Compare version strings. */
> > > extern int strverscmp (const char *, const char *);
>
>
> Grüße,
> Thomas
[-- Attachment #2: Type: application/pgp-signature, Size: 472 bytes --]
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Commit policy? Re: [PATCH 7/7] Plug ipa-prop escape analysis into gimple_call_arg_flags
2014-06-10 6:48 ` Commit policy? " Thomas Schwinge
@ 2014-06-10 6:54 ` Andrew Pinski
2014-06-10 9:07 ` Richard Biener
1 sibling, 0 replies; 10+ messages in thread
From: Andrew Pinski @ 2014-06-10 6:54 UTC (permalink / raw)
To: Thomas Schwinge
Cc: GCC Patches, Dominique Dhumieres, Jan Hubicka, Richard Guenther,
dave.anglin, ro, iains
On Mon, Jun 9, 2014 at 11:47 PM, Thomas Schwinge
<thomas@codesourcery.com> wrote:
> Hi!
>
> On Tue, 3 Jun 2014 11:55:44 +0200, I wrote:
>> Ping -- OK to commit to trunk?
>
> Even though several of those who I'd consider regular GCC developers do
> agree with this patch (see also <https://gcc.gnu.org/PR61334>), that is
> not enough "consensus" to consider this patch approved, right? Wouldn't
> it be a good idea for GCC to move to a more "liberal" policy in this
> regard? (That seems to work nicely for glibc.)
Really I think this is an obvious patch and would be the exact same
patch I would have came up with.
Thanks,
Andrew Pinski
>
>
> Ping.
>
>
>> On Wed, 28 May 2014 23:55:31 +0200, Jan Hubicka <hubicka@ucw.cz> wrote:
>> > > On Mon, 26 May 2014 02:16:35 -0700, Andrew Pinski <pinskia@gmail.com> wrote:
>> > > > On Mon, May 26, 2014 at 1:59 AM, Dominique Dhumieres <dominiq@lps.ens.fr> wrote:
>> > > > > r210901 breaks bootstrap on targets not supporting strnlen, e.g., darwin10.
>> > > > >
>> > > > > ../../_clean/gcc/lto-cgraph.c:976:68: error: 'strnlen' was not declared in this scope
>> > >
>> > > I'm seeing the same on MinGW, which also doesn't have strnlen (which is a
>> > > GNU extension).
>> > >
>> > > > strnlen should be declared in include/libiberty.h if there is no
>> > > > declaration as libiberty support is already there. That should be a
>> > > > simple fix.
>> > >
>> > > Like this?
>> >
>> > This looks good to me (thoguh strnlen is posix). I can not approve the patch
>> > but I would preffer it over just hand implementing strnlen there (that is easy,
>> > too)
>>
>> Patch is also considered good by testers in
>> <https://gcc.gnu.org/PR61334>.
>>
>> > > --- gcc/config.in
>> > > +++ gcc/config.in
>> > > [Regenerate.]
>> > > --- gcc/configure
>> > > +++ gcc/configure
>> > > [Regenerate.]
>> > > --- gcc/configure.ac
>> > > +++ gcc/configure.ac
>> > > @@ -1136,7 +1136,7 @@ CFLAGS="$CFLAGS -I${srcdir} -I${srcdir}/../include $GMPINC"
>> > > saved_CXXFLAGS="$CXXFLAGS"
>> > > CXXFLAGS="$CXXFLAGS -I${srcdir} -I${srcdir}/../include $GMPINC"
>> > > gcc_AC_CHECK_DECLS(getenv atol asprintf sbrk abort atof getcwd getwd \
>> > > - strsignal strstr stpcpy strverscmp \
>> > > + stpcpy strnlen strsignal strstr strverscmp \
>> > > errno snprintf vsnprintf vasprintf malloc realloc calloc \
>> > > free basename getopt clock getpagesize ffs gcc_UNLOCKED_FUNCS, , ,[
>> > > #include "ansidecl.h"
>> > > diff --git include/libiberty.h include/libiberty.h
>> > > index 7fd0703..56b8b43 100644
>> > > --- include/libiberty.h
>> > > +++ include/libiberty.h
>> > > @@ -636,6 +636,10 @@ extern int snprintf (char *, size_t, const char *, ...) ATTRIBUTE_PRINTF_3;
>> > > extern int vsnprintf (char *, size_t, const char *, va_list) ATTRIBUTE_PRINTF(3,0);
>> > > #endif
>> > >
>> > > +#if defined (HAVE_DECL_STRNLEN) && !HAVE_DECL_STRNLEN
>> > > +extern size_t strnlen (const char *, size_t);
>> > > +#endif
>> > > +
>> > > #if defined(HAVE_DECL_STRVERSCMP) && !HAVE_DECL_STRVERSCMP
>> > > /* Compare version strings. */
>> > > extern int strverscmp (const char *, const char *);
>>
>>
>> Grüße,
>> Thomas
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Commit policy? Re: [PATCH 7/7] Plug ipa-prop escape analysis into gimple_call_arg_flags
2014-06-10 6:48 ` Commit policy? " Thomas Schwinge
2014-06-10 6:54 ` Andrew Pinski
@ 2014-06-10 9:07 ` Richard Biener
1 sibling, 0 replies; 10+ messages in thread
From: Richard Biener @ 2014-06-10 9:07 UTC (permalink / raw)
To: Thomas Schwinge
Cc: GCC Patches, Dominique Dhumieres, Jan Hubicka, Andrew Pinski,
dave.anglin, ro, iains
On Tue, Jun 10, 2014 at 8:47 AM, Thomas Schwinge
<thomas@codesourcery.com> wrote:
> Hi!
>
> On Tue, 3 Jun 2014 11:55:44 +0200, I wrote:
>> Ping -- OK to commit to trunk?
>
> Even though several of those who I'd consider regular GCC developers do
> agree with this patch (see also <https://gcc.gnu.org/PR61334>), that is
> not enough "consensus" to consider this patch approved, right? Wouldn't
> it be a good idea for GCC to move to a more "liberal" policy in this
> regard? (That seems to work nicely for glibc.)
Ok.
Thanks,
Richard.
>
> Ping.
>
>
>> On Wed, 28 May 2014 23:55:31 +0200, Jan Hubicka <hubicka@ucw.cz> wrote:
>> > > On Mon, 26 May 2014 02:16:35 -0700, Andrew Pinski <pinskia@gmail.com> wrote:
>> > > > On Mon, May 26, 2014 at 1:59 AM, Dominique Dhumieres <dominiq@lps.ens.fr> wrote:
>> > > > > r210901 breaks bootstrap on targets not supporting strnlen, e.g., darwin10.
>> > > > >
>> > > > > ../../_clean/gcc/lto-cgraph.c:976:68: error: 'strnlen' was not declared in this scope
>> > >
>> > > I'm seeing the same on MinGW, which also doesn't have strnlen (which is a
>> > > GNU extension).
>> > >
>> > > > strnlen should be declared in include/libiberty.h if there is no
>> > > > declaration as libiberty support is already there. That should be a
>> > > > simple fix.
>> > >
>> > > Like this?
>> >
>> > This looks good to me (thoguh strnlen is posix). I can not approve the patch
>> > but I would preffer it over just hand implementing strnlen there (that is easy,
>> > too)
>>
>> Patch is also considered good by testers in
>> <https://gcc.gnu.org/PR61334>.
>>
>> > > --- gcc/config.in
>> > > +++ gcc/config.in
>> > > [Regenerate.]
>> > > --- gcc/configure
>> > > +++ gcc/configure
>> > > [Regenerate.]
>> > > --- gcc/configure.ac
>> > > +++ gcc/configure.ac
>> > > @@ -1136,7 +1136,7 @@ CFLAGS="$CFLAGS -I${srcdir} -I${srcdir}/../include $GMPINC"
>> > > saved_CXXFLAGS="$CXXFLAGS"
>> > > CXXFLAGS="$CXXFLAGS -I${srcdir} -I${srcdir}/../include $GMPINC"
>> > > gcc_AC_CHECK_DECLS(getenv atol asprintf sbrk abort atof getcwd getwd \
>> > > - strsignal strstr stpcpy strverscmp \
>> > > + stpcpy strnlen strsignal strstr strverscmp \
>> > > errno snprintf vsnprintf vasprintf malloc realloc calloc \
>> > > free basename getopt clock getpagesize ffs gcc_UNLOCKED_FUNCS, , ,[
>> > > #include "ansidecl.h"
>> > > diff --git include/libiberty.h include/libiberty.h
>> > > index 7fd0703..56b8b43 100644
>> > > --- include/libiberty.h
>> > > +++ include/libiberty.h
>> > > @@ -636,6 +636,10 @@ extern int snprintf (char *, size_t, const char *, ...) ATTRIBUTE_PRINTF_3;
>> > > extern int vsnprintf (char *, size_t, const char *, va_list) ATTRIBUTE_PRINTF(3,0);
>> > > #endif
>> > >
>> > > +#if defined (HAVE_DECL_STRNLEN) && !HAVE_DECL_STRNLEN
>> > > +extern size_t strnlen (const char *, size_t);
>> > > +#endif
>> > > +
>> > > #if defined(HAVE_DECL_STRVERSCMP) && !HAVE_DECL_STRVERSCMP
>> > > /* Compare version strings. */
>> > > extern int strverscmp (const char *, const char *);
>>
>>
>> Grüße,
>> Thomas
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2014-06-10 9:07 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-05-26 9:00 [PATCH 7/7] Plug ipa-prop escape analysis into gimple_call_arg_flags Dominique Dhumieres
2014-05-26 9:16 ` Andrew Pinski
2014-05-26 9:29 ` Christophe Lyon
2014-05-26 16:32 ` Jan Hubicka
2014-05-28 7:26 ` Thomas Schwinge
2014-05-28 21:55 ` Jan Hubicka
2014-06-03 9:56 ` Thomas Schwinge
2014-06-10 6:48 ` Commit policy? " Thomas Schwinge
2014-06-10 6:54 ` Andrew Pinski
2014-06-10 9:07 ` Richard Biener
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).