public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c++/41020]  New: Can't declare an extern "C" friend.
@ 2009-08-09 21:30 davek at gcc dot gnu dot org
  2009-08-09 21:32 ` [Bug c++/41020] " davek at gcc dot gnu dot org
                   ` (16 more replies)
  0 siblings, 17 replies; 19+ messages in thread
From: davek at gcc dot gnu dot org @ 2009-08-09 21:30 UTC (permalink / raw)
  To: gcc-bugs

Attempting to declare a previously-declared extern "C" function as a friend
within a class definition fails with an error that the friend declaration
ambiguates the original extern "C" declaration.  It appears that the friend
declaration is taken as an overload with "C++" linkage, since G++ then
complains that the class members are private within the context of the extern
"C" function definition.

$ cat friend.cxx

extern "C"
{
  int fork (void);
};

class frok
{
  int this_errno;
  friend int fork (void);
};

extern "C" int
fork (void)
{
  frok grouped;
  return grouped.this_errno;
}

$ g++-4 -c friend.cxx -o friend.o
friend.cxx:10:24: error: new declaration 'int fork()'
friend.cxx:4:7: error: ambiguates old declaration 'int fork()'
friend.cxx: In function 'int fork()':
friend.cxx:9:7: error: 'int frok::this_errno' is private
friend.cxx:17:18: error: within this context

$

  According to, for example, "C++ in a nutshell"
(http://books.google.co.uk/books?id=91JTA9B_m44C&pg=PA170&lpg=PA170&dq=friend+%22storage+class%22&source=bl&ots=HrN4X1Y5wu&sig=N9rnB8r_YnxbH2hiWGqtbAWbWyk&hl=en&ei=oDt_SqmvGJKNjAe9k93wAQ&sa=X&oi=book_result&ct=result&resnum=5#v=onepage&q=friend%20%22storage%20class%22&f=false),
a function in a friend declaration should "retain its original linkage" when it
has already been declared.  That is referring to the case of applying a storage
class specifier (static) to the prior declaration that isn't syntactically
valid within a friend declaration, but it seems like it should apply here too;
the linkage should still be retained, even though I'm not changing the
storage-class specifier.

  There may be some reason why this isn't valid C++, but I think it's probably
a weakness in the parsing of friend declarations.

$ g++-4 -v
Using built-in specs.
Target: i686-pc-cygwin
Configured with: /gnu/gcc/gcc-patched/configure --prefix=/opt/gcc-tools -v
--wit
h-gmp=/usr --with-mpfr=/usr --enable-bootstrap
--enable-version-specific-runtime
-libs --enable-static --enable-shared --enable-shared-libgcc
--disable-__cxa_ate
xit --with-gnu-ld --with-gnu-as --with-dwarf2 --disable-sjlj-exceptions
--disabl
e-symvers --disable-libjava --disable-interpreter --program-suffix=-4
--disable-
libgomp --enable-libssp --enable-libada --enable-threads=posix --with-arch=i686
--with-tune=generic CC=gcc-4 CXX=g++-4 CC_FOR_TARGET=gcc-4 CXX_FOR_TARGET=g++-4
--with-ecj-jar=/usr/share/java/ecj.jar LD=/opt/gcc-tools/bin/ld.exe
LD_FOR_TARGE
T=/opt/gcc-tools/bin/ld.exe AS=/opt/gcc-tools/bin/as.exe
AS_FOR_TARGET=/opt/gcc-
tools/bin/as.exe --disable-win32-registry --disable-libgcj-debug
--enable-langua
ges=c,c++,ada
Thread model: posix
gcc version 4.5.0 20090730 (experimental) (GCC)


-- 
           Summary: Can't declare an extern "C" friend.
           Product: gcc
           Version: 4.5.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: davek at gcc dot gnu dot org
 GCC build triplet: i686-pc-cygwin
  GCC host triplet: i686-pc-cygwin
GCC target triplet: i686-pc-cygwin


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] Can't declare an extern "C" friend.
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
@ 2009-08-09 21:32 ` davek at gcc dot gnu dot org
  2009-08-10 10:40 ` jwakely dot gcc at gmail dot com
                   ` (15 subsequent siblings)
  16 siblings, 0 replies; 19+ messages in thread
From: davek at gcc dot gnu dot org @ 2009-08-09 21:32 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #1 from davek at gcc dot gnu dot org  2009-08-09 21:31 -------
(In reply to comment #0)
> $ g++-4 -v

  Yuck, that got horribly wrapped.  Here it is again, hopefully formatted a
little better this time:

Using built-in specs.
Target: i686-pc-cygwin
Configured with: /gnu/gcc/gcc-patched/configure --prefix=/opt/gcc-tools -v
--with-gmp=/usr --with-mpfr=/usr --enable-bootstrap
--enable-version-specific-runtime-libs --enable-static --enable-shared
--enable-shared-libgcc --disable-__cxa_atexit --with-gnu-ld --with-gnu-as
--with-dwarf2 --disable-sjlj-exceptions --disable-symvers --disable-libjava
--disable-interpreter --program-suffix=-4 --disable-libgomp --enable-libssp
--enable-libada --enable-threads=posix --with-arch=i686 --with-tune=generic
CC=gcc-4 CXX=g++-4 CC_FOR_TARGET=gcc-4 CXX_FOR_TARGET=g++-4
--with-ecj-jar=/usr/share/java/ecj.jar LD=/opt/gcc-tools/bin/ld.exe
LD_FOR_TARGET=/opt/gcc-tools/bin/ld.exe AS=/opt/gcc-tools/bin/as.exe
AS_FOR_TARGET=/opt/gcc-tools/bin/as.exe --disable-win32-registry
--disable-libgcj-debug --enable-languages=c,c++,ada
Thread model: posix
gcc version 4.5.0 20090730 (experimental) (GCC) 


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] Can't declare an extern "C" friend.
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
  2009-08-09 21:32 ` [Bug c++/41020] " davek at gcc dot gnu dot org
@ 2009-08-10 10:40 ` jwakely dot gcc at gmail dot com
  2009-08-10 12:45 ` [Bug c++/41020] [4.5 Regression] " rguenth at gcc dot gnu dot org
                   ` (14 subsequent siblings)
  16 siblings, 0 replies; 19+ messages in thread
From: jwakely dot gcc at gmail dot com @ 2009-08-10 10:40 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #2 from jwakely dot gcc at gmail dot com  2009-08-10 10:40 -------
Apart from the semi-colon after the extern "C" block the code is valid and this
is a recent regression on trunk.


-- 

jwakely dot gcc at gmail dot com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jwakely dot gcc at gmail dot
                   |                            |com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend.
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
  2009-08-09 21:32 ` [Bug c++/41020] " davek at gcc dot gnu dot org
  2009-08-10 10:40 ` jwakely dot gcc at gmail dot com
@ 2009-08-10 12:45 ` rguenth at gcc dot gnu dot org
  2009-08-10 16:17 ` davek at gcc dot gnu dot org
                   ` (13 subsequent siblings)
  16 siblings, 0 replies; 19+ messages in thread
From: rguenth at gcc dot gnu dot org @ 2009-08-10 12:45 UTC (permalink / raw)
  To: gcc-bugs



-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |rejects-valid
            Summary|Can't declare an extern "C" |[4.5 Regression] Can't
                   |friend.                     |declare an extern "C"
                   |                            |friend.
   Target Milestone|---                         |4.5.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend.
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
                   ` (2 preceding siblings ...)
  2009-08-10 12:45 ` [Bug c++/41020] [4.5 Regression] " rguenth at gcc dot gnu dot org
@ 2009-08-10 16:17 ` davek at gcc dot gnu dot org
  2009-08-10 17:05 ` jwakely dot gcc at gmail dot com
                   ` (12 subsequent siblings)
  16 siblings, 0 replies; 19+ messages in thread
From: davek at gcc dot gnu dot org @ 2009-08-10 16:17 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #3 from davek at gcc dot gnu dot org  2009-08-10 16:17 -------
(In reply to comment #2)
> Apart from the semi-colon after the extern "C" block the code is valid and this
> is a recent regression on trunk.

I am fairly sure that a semi-colon after a block statement like that is
unnecessary, but not actually invalid.  It certainly makes no difference to the
testcase whether it is present or absent.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend.
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
                   ` (3 preceding siblings ...)
  2009-08-10 16:17 ` davek at gcc dot gnu dot org
@ 2009-08-10 17:05 ` jwakely dot gcc at gmail dot com
  2009-08-10 17:16 ` davek at gcc dot gnu dot org
                   ` (11 subsequent siblings)
  16 siblings, 0 replies; 19+ messages in thread
From: jwakely dot gcc at gmail dot com @ 2009-08-10 17:05 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #4 from jwakely dot gcc at gmail dot com  2009-08-10 17:05 -------
It's irrelevant to this bug and is just me being more pedantic than -pedantic,
however ... even with -pedantic GCC has always accepted stray semi-colons at
namespace scope, but it's not valid.

At function scope a lone ';' is a valid expression-statement, but
expression-statements are not allowed at namespace scope, only declarations
are, and ';' is not a valid declaration.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend.
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
                   ` (4 preceding siblings ...)
  2009-08-10 17:05 ` jwakely dot gcc at gmail dot com
@ 2009-08-10 17:16 ` davek at gcc dot gnu dot org
  2009-08-24  1:00 ` mmitchel at gcc dot gnu dot org
                   ` (10 subsequent siblings)
  16 siblings, 0 replies; 19+ messages in thread
From: davek at gcc dot gnu dot org @ 2009-08-10 17:16 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #5 from davek at gcc dot gnu dot org  2009-08-10 17:16 -------
(In reply to comment #4)
> It's irrelevant to this bug and is just me being more pedantic than -pedantic,
> however ... even with -pedantic GCC has always accepted stray semi-colons at
> namespace scope, but it's not valid.
> 
> At function scope a lone ';' is a valid expression-statement, but
> expression-statements are not allowed at namespace scope, only declarations
> are, and ';' is not a valid declaration.

  Well you learn something new every day!  Never realised that was a gnu
extension, but it sure is, comeau online choked on my testcase:

"ComeauTest.c", line 5: error: extra ";" ignored,
        In C: A function definition does not end with a semicolon
        In C++: A non-member function definition, extern "C" block,
                or namespace does not end with a semicolon
  };
   ^
  Anyway.  Consider the testcase amended.  :-)

$ cat friend.cxx

extern "C"
{
  int fork (void);
}

class frok
{
  int this_errno;
  friend int fork (void);
};

extern "C" int
fork (void)
{
  frok grouped;
  return grouped.this_errno;
}

$ g++-4 -c friend.cxx -o friend.o
friend.cxx:10:24: error: new declaration 'int fork()'
friend.cxx:4:7: error: ambiguates old declaration 'int fork()'
friend.cxx: In function 'int fork()':
friend.cxx:9:7: error: 'int frok::this_errno' is private
friend.cxx:17:18: error: within this context

$


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend.
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
                   ` (5 preceding siblings ...)
  2009-08-10 17:16 ` davek at gcc dot gnu dot org
@ 2009-08-24  1:00 ` mmitchel at gcc dot gnu dot org
  2009-09-18  3:56 ` pinskia at gcc dot gnu dot org
                   ` (9 subsequent siblings)
  16 siblings, 0 replies; 19+ messages in thread
From: mmitchel at gcc dot gnu dot org @ 2009-08-24  1:00 UTC (permalink / raw)
  To: gcc-bugs



-- 

mmitchel at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend.
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
                   ` (6 preceding siblings ...)
  2009-08-24  1:00 ` mmitchel at gcc dot gnu dot org
@ 2009-09-18  3:56 ` pinskia at gcc dot gnu dot org
  2009-10-04 19:14 ` [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend of a builtin function pinskia at gcc dot gnu dot org
                   ` (8 subsequent siblings)
  16 siblings, 0 replies; 19+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2009-09-18  3:56 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #6 from pinskia at gcc dot gnu dot org  2009-09-18 03:56 -------
Confirmed.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2009-09-18 03:56:23
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend of a builtin function
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
                   ` (7 preceding siblings ...)
  2009-09-18  3:56 ` pinskia at gcc dot gnu dot org
@ 2009-10-04 19:14 ` pinskia at gcc dot gnu dot org
  2009-10-23 18:19   ` Dodji Seketeli
  2009-10-23 18:20 ` dodji at redhat dot com
                   ` (7 subsequent siblings)
  16 siblings, 1 reply; 19+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2009-10-04 19:14 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #7 from pinskia at gcc dot gnu dot org  2009-10-04 19:13 -------
This is related to builtins too because either using -fno-builtin or using
fork1 allows this program to work.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[4.5 Regression] Can't      |[4.5 Regression] Can't
                   |declare an extern "C"       |declare an extern "C" friend
                   |friend.                     |of a builtin function


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* Re: [Bug c++/41020] [4.5 Regression] Can't declare an extern "C"  friend of a builtin function
  2009-10-04 19:14 ` [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend of a builtin function pinskia at gcc dot gnu dot org
@ 2009-10-23 18:19   ` Dodji Seketeli
  0 siblings, 0 replies; 19+ messages in thread
From: Dodji Seketeli @ 2009-10-23 18:19 UTC (permalink / raw)
  To: pinskia at gcc dot gnu dot org; +Cc: gcc-bugs

Indeed. I am testing the patch below.

diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c
index 5eb389f..7c01ee2 100644
--- a/gcc/cp/decl.c
+++ b/gcc/cp/decl.c
@@ -104,6 +104,7 @@ static void store_parm_decls (tree);
 static void initialize_local_var (tree, tree);
 static void expand_static_init (tree, tree);
 static tree next_initializable_field (tree);
+static int decls_match_1 (tree, tree, bool);
 
 /* The following symbols are subsumed in the cp_global_trees array, and
    listed here individually for documentation purposes.
@@ -899,6 +900,14 @@ push_local_name (tree decl)
 int
 decls_match (tree newdecl, tree olddecl)
 {
+  return decls_match_1 (newdecl, olddecl, /* newdecl_is_friend  */ false);
+}
+
+/* Subroutine of decls_match.  */
+
+static int
+decls_match_1 (tree newdecl, tree olddecl, bool newdecl_is_friend)
+{
   int types_match;
 
   if (newdecl == olddecl)
@@ -934,9 +943,11 @@ decls_match (tree newdecl, tree olddecl)
 
 #ifdef NO_IMPLICIT_EXTERN_C
       /* A new declaration doesn't match a built-in one unless it
-	 is also extern "C".  */
+	 is also extern "C". Friend function re-declarations retain the
+	 the linkage of the original declaration though.  */
       if (DECL_BUILT_IN (olddecl)
-	  && DECL_EXTERN_C_P (olddecl) && !DECL_EXTERN_C_P (newdecl))
+	  && DECL_EXTERN_C_P (olddecl) && !DECL_EXTERN_C_P (newdecl)
+	  && !newdecl_is_friend)
 	return 0;
 #endif
 
@@ -1122,7 +1133,7 @@ duplicate_decls (tree newdecl, tree olddecl, bool newdecl_is_friend)
   if (newdecl == olddecl)
     return olddecl;
 
-  types_match = decls_match (newdecl, olddecl);
+  types_match = decls_match_1 (newdecl, olddecl, newdecl_is_friend);
 
   /* If either the type of the new decl or the type of the old decl is an
      error_mark_node, then that implies that we have already issued an


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend of a builtin function
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
                   ` (8 preceding siblings ...)
  2009-10-04 19:14 ` [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend of a builtin function pinskia at gcc dot gnu dot org
@ 2009-10-23 18:20 ` dodji at redhat dot com
  2009-10-23 18:34 ` dodji at gcc dot gnu dot org
                   ` (6 subsequent siblings)
  16 siblings, 0 replies; 19+ messages in thread
From: dodji at redhat dot com @ 2009-10-23 18:20 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #8 from dodji at gcc dot gnu dot org  2009-10-23 18:19 -------
Subject: Re:  [4.5 Regression] Can't declare an extern "C"
        friend of a builtin function

Indeed. I am testing the patch below.

diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c
index 5eb389f..7c01ee2 100644
--- a/gcc/cp/decl.c
+++ b/gcc/cp/decl.c
@@ -104,6 +104,7 @@ static void store_parm_decls (tree);
 static void initialize_local_var (tree, tree);
 static void expand_static_init (tree, tree);
 static tree next_initializable_field (tree);
+static int decls_match_1 (tree, tree, bool);

 /* The following symbols are subsumed in the cp_global_trees array, and
    listed here individually for documentation purposes.
@@ -899,6 +900,14 @@ push_local_name (tree decl)
 int
 decls_match (tree newdecl, tree olddecl)
 {
+  return decls_match_1 (newdecl, olddecl, /* newdecl_is_friend  */ false);
+}
+
+/* Subroutine of decls_match.  */
+
+static int
+decls_match_1 (tree newdecl, tree olddecl, bool newdecl_is_friend)
+{
   int types_match;

   if (newdecl == olddecl)
@@ -934,9 +943,11 @@ decls_match (tree newdecl, tree olddecl)

 #ifdef NO_IMPLICIT_EXTERN_C
       /* A new declaration doesn't match a built-in one unless it
-        is also extern "C".  */
+        is also extern "C". Friend function re-declarations retain the
+        the linkage of the original declaration though.  */
       if (DECL_BUILT_IN (olddecl)
-         && DECL_EXTERN_C_P (olddecl) && !DECL_EXTERN_C_P (newdecl))
+         && DECL_EXTERN_C_P (olddecl) && !DECL_EXTERN_C_P (newdecl)
+         && !newdecl_is_friend)
        return 0;
 #endif

@@ -1122,7 +1133,7 @@ duplicate_decls (tree newdecl, tree olddecl, bool
newdecl_is_friend)
   if (newdecl == olddecl)
     return olddecl;

-  types_match = decls_match (newdecl, olddecl);
+  types_match = decls_match_1 (newdecl, olddecl, newdecl_is_friend);

   /* If either the type of the new decl or the type of the old decl is an
      error_mark_node, then that implies that we have already issued an


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend of a builtin function
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
                   ` (9 preceding siblings ...)
  2009-10-23 18:20 ` dodji at redhat dot com
@ 2009-10-23 18:34 ` dodji at gcc dot gnu dot org
  2009-10-23 19:32 ` dodji at redhat dot com
                   ` (5 subsequent siblings)
  16 siblings, 0 replies; 19+ messages in thread
From: dodji at gcc dot gnu dot org @ 2009-10-23 18:34 UTC (permalink / raw)
  To: gcc-bugs



-- 

dodji at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |dodji at gcc dot gnu dot org
                   |dot org                     |
             Status|NEW                         |ASSIGNED
   Last reconfirmed|2009-09-18 03:56:23         |2009-10-23 18:34:28
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend of a builtin function
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
                   ` (10 preceding siblings ...)
  2009-10-23 18:34 ` dodji at gcc dot gnu dot org
@ 2009-10-23 19:32 ` dodji at redhat dot com
  2009-10-26 14:40 ` dodji at gcc dot gnu dot org
                   ` (4 subsequent siblings)
  16 siblings, 0 replies; 19+ messages in thread
From: dodji at redhat dot com @ 2009-10-23 19:32 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #9 from dodji at gcc dot gnu dot org  2009-10-23 19:32 -------
Subject: Re:   New: Can't declare an extern "C" friend.

Patch submitted to http://gcc.gnu.org/ml/gcc-patches/2009-10/msg01486.html


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend of a builtin function
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
                   ` (11 preceding siblings ...)
  2009-10-23 19:32 ` dodji at redhat dot com
@ 2009-10-26 14:40 ` dodji at gcc dot gnu dot org
  2009-10-26 16:06 ` dodji at gcc dot gnu dot org
                   ` (3 subsequent siblings)
  16 siblings, 0 replies; 19+ messages in thread
From: dodji at gcc dot gnu dot org @ 2009-10-26 14:40 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #10 from dodji at gcc dot gnu dot org  2009-10-26 14:40 -------
Subject: Bug 41020

Author: dodji
Date: Mon Oct 26 14:40:16 2009
New Revision: 153552

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=153552
Log:
Fix PR c++/41020

gcc/cp/ChangeLog:

        PR c++/41020
        * decl.c (decls_match): Use DECL_IS_BUILTIN instead of
        DECL_BUILT_IN.

gcc/testsuite/ChangeLog:
        PR c++/41020
        * g++.dg/lookup/extern-c-redecl2.C: New test.
        * g++.dg/lookup/extern-c-redecl3.C: Likewise.
        * g++.dg/lookup/extern-c-redecl4.C: Likewise.
        * g++.dg/lookup/extern-c-redecl5.C: Likewise.

Added:
    trunk/gcc/testsuite/g++.dg/lookup/extern-c-redecl2.C
    trunk/gcc/testsuite/g++.dg/lookup/extern-c-redecl3.C
    trunk/gcc/testsuite/g++.dg/lookup/extern-c-redecl4.C
    trunk/gcc/testsuite/g++.dg/lookup/extern-c-redecl5.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/decl.c
    trunk/gcc/testsuite/ChangeLog


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend of a builtin function
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
                   ` (12 preceding siblings ...)
  2009-10-26 14:40 ` dodji at gcc dot gnu dot org
@ 2009-10-26 16:06 ` dodji at gcc dot gnu dot org
  2009-10-27 10:29 ` jakub at gcc dot gnu dot org
                   ` (2 subsequent siblings)
  16 siblings, 0 replies; 19+ messages in thread
From: dodji at gcc dot gnu dot org @ 2009-10-26 16:06 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #11 from dodji at gcc dot gnu dot org  2009-10-26 16:06 -------
Fixed in 4.5.0


-- 

dodji at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend of a builtin function
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
                   ` (13 preceding siblings ...)
  2009-10-26 16:06 ` dodji at gcc dot gnu dot org
@ 2009-10-27 10:29 ` jakub at gcc dot gnu dot org
  2009-10-29 16:53 ` law at gcc dot gnu dot org
  2009-10-30 16:07 ` hjl at gcc dot gnu dot org
  16 siblings, 0 replies; 19+ messages in thread
From: jakub at gcc dot gnu dot org @ 2009-10-27 10:29 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #12 from jakub at gcc dot gnu dot org  2009-10-27 10:29 -------
Subject: Bug 41020

Author: jakub
Date: Tue Oct 27 10:28:48 2009
New Revision: 153587

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=153587
Log:
        PR c++/41020
        * g++.dg/lookup/extern-c-redecl5.C: Fix up regexp.

Modified:
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/g++.dg/lookup/extern-c-redecl5.C


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend of a builtin function
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
                   ` (14 preceding siblings ...)
  2009-10-27 10:29 ` jakub at gcc dot gnu dot org
@ 2009-10-29 16:53 ` law at gcc dot gnu dot org
  2009-10-30 16:07 ` hjl at gcc dot gnu dot org
  16 siblings, 0 replies; 19+ messages in thread
From: law at gcc dot gnu dot org @ 2009-10-29 16:53 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #13 from law at gcc dot gnu dot org  2009-10-29 16:50 -------
Subject: Bug 41020

Author: law
Date: Thu Oct 29 16:48:00 2009
New Revision: 153715

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=153715
Log:
Recorded merge of revisions
153580-153581,153584,153586-153600,153604,153606,153610,153613,153615-153618,153621,153643,153646-153648,153650-153652,153654-153667,153669-153671
via svnmerge from 
svn+ssh://law@gcc.gnu.org/svn/gcc/trunk

........
  r153580 | gccadmin | 2009-10-26 18:17:26 -0600 (Mon, 26 Oct 2009) | 1 line

  Daily bump.
........
  r153581 | paolo | 2009-10-26 19:18:10 -0600 (Mon, 26 Oct 2009) | 6 lines

  2009-10-26  Paolo Carlini  <paolo.carlini@oracle.com>

        * include/std/chrono (duration<>::duration(const duration<>&)): Fix
        per the straightforward resolution of DR 974.
        * testsuite/20_util/duration/cons/dr974.cc: Add.
........
  r153584 | carrot | 2009-10-27 03:06:36 -0600 (Tue, 27 Oct 2009) | 16 lines

        * target.h (have_conditional_execution): Add a new target hook
function.
        * target-def.h (TARGET_HAVE_CONDITIONAL_EXECUTION): Likewise.
        * targhooks.h (default_have_conditional_execution): Likewise.
        * targhooks.c (default_have_conditional_execution): Likewise.
        * doc/tm.texi (TARGET_HAVE_CONDITIONAL_EXECUTION): Document it.
        * config/arm/arm.c (TARGET_HAVE_CONDITIONAL_EXECUTION): Define it.
        (arm_have_conditional_execution): New function.
        * ifcvt.c (noce_process_if_block, find_if_header,
        cond_exec_find_if_block, dead_or_predicable): Change the usage of macro
        HAVE_conditional_execution to a target hook call.
        * recog.c (peephole2_optimize): Likewise.
        * sched-rgn.c (add_branch_dependences): Likewise.
        * final.c (asm_insn_count, final_scan_insn): Likewise.
        * bb-reorder.c (HAVE_conditional_execution): Remove it.
........
  r153586 | ebotcazou | 2009-10-27 04:09:04 -0600 (Tue, 27 Oct 2009) | 1 line

  Fix nits
........
  r153587 | jakub | 2009-10-27 04:28:48 -0600 (Tue, 27 Oct 2009) | 3 lines

        PR c++/41020
        * g++.dg/lookup/extern-c-redecl5.C: Fix up regexp.
........
  r153588 | aldyh | 2009-10-27 05:18:12 -0600 (Tue, 27 Oct 2009) | 5 lines

        PR bootstrap/41451
        * fold-const.c (fold_binary_loc): Do not call
        protected_set_expr_location.
........
  r153589 | rguenth | 2009-10-27 05:30:59 -0600 (Tue, 27 Oct 2009) | 5 lines

  2009-10-27  Richard Guenther  <rguenther@suse.de>

        PR lto/41821
        * gimple.c (gimple_types_compatible_p): Handle OFFSET_TYPE.
........
  r153590 | revitale | 2009-10-27 05:46:07 -0600 (Tue, 27 Oct 2009) | 1 line

  Fix PR40648 -- Fix misaligned store vectorizer patch
........
  r153591 | charlet | 2009-10-27 07:06:06 -0600 (Tue, 27 Oct 2009) | 16 lines

  2009-10-27  Arnaud Charlet  <charlet@adacore.com>

        * exp_aggr.adb: Fix comment.

  2009-10-27  Emmanuel Briot  <briot@adacore.com>

        * prj-err.adb (Error_Msg): take into account continuation lines when
        computing whether we have a warning.

  2009-10-27  Vasiliy Fofanov  <fofanov@adacore.com>

        * make.adb, s-os_lib.adb, s-os_lib.ads (Create_Temp_Output_File): New
        routine that is designed to create temp file descriptor specifically
        for redirecting an output stream.
........
  r153592 | charlet | 2009-10-27 07:16:48 -0600 (Tue, 27 Oct 2009) | 45 lines

  2009-10-27  Vincent Celier  <celier@adacore.com>

        * makeutl.adb (Check_Source_Info_In_ALI): Do not recompile if a subunit
        from the runtime is found, except if gnatmake switch -a is used and
this
        subunit cannot be found.

  2009-10-27  Ed Schonberg  <schonberg@adacore.com>

        * gnatbind.adb (gnatbind): When the -R option is selected, list
subunits
        as well, for tools that need the complete closure of the main program.

  2009-10-27  Sergey Rybin  <rybin@adacore.com>

        * gnat_ugn.texi: Minor updates.

  2009-10-27  Emmanuel Briot  <briot@adacore.com>

        * prj-tree.adb (Free): Fix memory leak.

  2009-10-27  Vasiliy Fofanov  <fofanov@adacore.com>

        * adaint.c, s-os_lib.adb (__gnat_create_output_file_new): New function
        that ensures the file that is created is new. Use this function to make
        sure there is no race condition if several processes are creating temp
        files concurrently.

        * s-os_lib.ads: Update comment.

  2009-10-27  Thomas Quinot  <quinot@adacore.com>

        * sem_ch12.adb: Minor reformatting

  2009-10-27  Javier Miranda  <miranda@adacore.com>

        * exp_ch4.ads (Integer_Promotion_Possible): New subprogram.
        * exp_ch4.adb (Integer_Promotion_Possible): New subprogram.
        (Expand_N_Type_Conversion): Replace code that checks if the integer
        promotion of the operands is possible by a call to the new function
        Integer_Promotion_Possible. Minor reformating because an enclosing
        block is now not needed.
        * checks.adb (Apply_Arithmetic_Overflow_Check): Add missing check to
        see if the integer promotion is possible; in such case the runtime
        checks are not generated.
........
  r153593 | charlet | 2009-10-27 07:22:25 -0600 (Tue, 27 Oct 2009) | 17 lines

  2009-10-27  Thomas Quinot  <quinot@adacore.com>

        * sem_ch12.adb (Install_Formal_Packages): Do not omit installation of
        visible entities when the formal package doesn't have a box.

        * checks.adb: Minor reformatting.

  2009-10-27  Vincent Celier  <celier@adacore.com>

        * prj-part.adb (Parse): Catch exception Types.Unrecoverable_Error and
        set Project to Empty_Node.

  2009-10-27  Robert Dewar  <dewar@adacore.com>

        * gnatbind.adb: Minor reformatting
........
  r153594 | charlet | 2009-10-27 07:51:46 -0600 (Tue, 27 Oct 2009) | 18 lines

  2009-10-27  Robert Dewar  <dewar@adacore.com>

        * s-os_lib.ads, s-os_lib.adb, prj-err.adb, makeutl.adb: Minor
        reformatting.

  2009-10-27  Ed Schonberg  <schonberg@adacore.com>

        * sem.util.ads, sem_util.adb (Denotes_Same_Object,
        Denotes_Same_Prefix): New functions to detect overlap between actuals
        that are not by-copy in a call, when one of them is in-out.
        * sem_warn.ads, sem_warn.adb (Warn_On_Overlapping_Actuals): New
        procedure,  called on a subprogram call to warn when an in-out actual
        that is not by-copy overlaps with another actual, thus leadind to
        potentially dangerous aliasing in the body of the called subprogram.
        Currently the warning is under control of the -gnatX switch.
        * sem_res.adb (resolve_call): call Warn_On_Overlapping_Actuals.
........
  r153595 | charlet | 2009-10-27 08:02:58 -0600 (Tue, 27 Oct 2009) | 6 lines

  2009-10-27  Robert Dewar  <dewar@adacore.com>

        * sem_warn.adb, sem_util.adb, sem_util.ads: Minor reformatting. Add
        comments.
........
  r153596 | charlet | 2009-10-27 08:07:19 -0600 (Tue, 27 Oct 2009) | 2 lines

  Minor doc updates.
........
  r153597 | charlet | 2009-10-27 08:14:44 -0600 (Tue, 27 Oct 2009) | 6 lines

  2009-10-27  Robert Dewar  <dewar@adacore.com>

        * s-fileio.adb, s-fileio.ads, sem_util.adb, sem_warn.adb,
        sem_warn.ads: Minor reformatting
........
  r153598 | rguenth | 2009-10-27 09:16:35 -0600 (Tue, 27 Oct 2009) | 5 lines

  2009-10-27  Richard Guenther  <rguenther@suse.de>

        * tree-complex.c (expand_complex_div_wide): Check for
        INTEGER_CST, not TREE_CONSTANT on comparison folding result.
........
  r153599 | jakub | 2009-10-27 09:50:50 -0600 (Tue, 27 Oct 2009) | 6 lines

        PR c/41842
        * c-typeck.c (convert_arguments): Return -1 if any of the arguments is
        error_mark_node.

        * gcc.dg/pr41842.c: New test.
........
  r153600 | rguenth | 2009-10-27 09:52:44 -0600 (Tue, 27 Oct 2009) | 14 lines

  2009-10-27  Richard Guenther  <rguenther@suse.de>

        * tree-ssa-structalias.c (find_func_aliases): In IPA mode
        handle calls to externally visible functions like in regular mode.
        (create_variable_info_for): Do not create function infos here.
        (have_alias_info): Remove write-only variable.
        (solve_constraints): New function split out from common code
        in compute_points_to_sets and ipa_pta_execute.
        (compute_points_to_sets): Adjust.
        (ipa_pta_execute): Likewise.  Handle clones and externally visible
        functions like in non-IPA mode.

        * gcc.dg/torture/ipa-pta-1.c: Adjust testcase.
........
  r153604 | uros | 2009-10-27 11:03:47 -0600 (Tue, 27 Oct 2009) | 3 lines

        * ChangeLog: Fix formatting.
        * testsuite/ChangeLog: Ditto.
........
  r153606 | ktietz | 2009-10-27 11:14:47 -0600 (Tue, 27 Oct 2009) | 11 lines

  2009-10-27  Kai Tietz <kai.tietz@onevision.com>

          PR/41799
          * config/i386/mingw32.h (CHECK_EXECUTE_STACK_ENABLED): New macro.
          * config/i386/mingw.opt: Add fset-stack-executable.
          * config/i386/i386.c (ix86_trampoline_init): Make call to
          emit_library_call conditional, if CHECK_EXECUTE_STACK_ENABLED is
          defined and its value is not zero.
          * doc/invoke.texi
........
  r153610 | espindola | 2009-10-27 12:17:13 -0600 (Tue, 27 Oct 2009) | 7 lines

  2009-10-27  Dmitry Gorbachev  <d.g.gorbachev@gmail.com>

        PR lto/41652
        * configure.ac: Call AC_SYS_LARGEFILE before AC_OUTPUT.
        * configure: Regenerate.
........
  r153613 | ebotcazou | 2009-10-27 13:41:13 -0600 (Tue, 27 Oct 2009) | 4 lines

        * raise-gcc (db_region_for): Use _Unwind_GetIPInfo instead of
        _Unwind_GetIP if HAVE_GETIPINFO is defined.
        (db_action_for): Likewise.
........
  r153615 | rth | 2009-10-27 14:09:07 -0600 (Tue, 27 Oct 2009) | 7 lines

          PR c++/41819
          * tree-eh.c (eh_region_may_contain_throw_map): Rename from
          eh_region_may_contain_throw; update users.
          (eh_region_may_contain_throw): New function.
          (lower_catch): Check flag_exceptions before creating exception
region.
          (lower_eh_filter, lower_eh_must_not_throw): Likewise.
          (lower_cleanup): Tidy existing flag_exceptions check to match.
........
  r153616 | ebotcazou | 2009-10-27 14:24:31 -0600 (Tue, 27 Oct 2009) | 3 lines

        * gcc-interface/decl.c (purpose_member_field): New static function.
        (annotate_rep): Use it instead of purpose_member.
........
  r153617 | jason | 2009-10-27 15:58:09 -0600 (Tue, 27 Oct 2009) | 10 lines

        Allow no-capture lambdas to convert to function pointer.
        * semantics.c (maybe_add_lambda_conv_op): New.
        * parser.c (cp_parser_lambda_expression): Call it.
        (cp_parser_lambda_declarator_opt): Make op() static if
        no captures.
        * mangle.c (write_closure_type_name): Adjust.
        * semantics.c (finish_this_expr): Adjust.
        * decl.c (grok_op_properties): Allow it.
        * call.c (build_user_type_conversion_1): Handle static conversion op.
        (build_op_call): And op().
........
  r153618 | rth | 2009-10-27 17:25:54 -0600 (Tue, 27 Oct 2009) | 1 line

          * cgraphunit.c (cgraph_optimize): Maintain timevar stack properly.
........
  r153621 | gccadmin | 2009-10-27 18:16:59 -0600 (Tue, 27 Oct 2009) | 1 line

  Daily bump.
........
  r153643 | kkojima | 2009-10-27 22:22:21 -0600 (Tue, 27 Oct 2009) | 4 lines

        * config/sh/sh.md (stuff_delay_slot): Move const_int pattern
        inside the unspec vector.
........
  r153646 | bonzini | 2009-10-28 03:49:58 -0600 (Wed, 28 Oct 2009) | 6 lines

  2009-10-28  Paolo Bonzini  <bonzini@gnu.org>

        * config/sh/sh.md (cbranchfp4_media): Remove hack extending
        cstore result to DImode.
........
  r153647 | bonzini | 2009-10-28 03:54:01 -0600 (Wed, 28 Oct 2009) | 6 lines

  2009-10-28  Paolo Bonzini  <bonzini@gnu.org>

        * expmed.c (emit_store_flag): Check costs before
        transforming to the opposite representation.
........
  r153648 | singler | 2009-10-28 04:04:03 -0600 (Wed, 28 Oct 2009) | 8 lines

  2009-10-28  Johannes Singler  <singler@kit.edu>

          PR libstdc++/40852
          * include/parallel/multiseq_selection.h
          (multiseq_partition, multiseq_selection):  Avoid intermediate
          values exceeding the integer type range for very large inputs.
........
  r153650 | bonzini | 2009-10-28 04:17:29 -0600 (Wed, 28 Oct 2009) | 15 lines

  2009-10-28  Paolo Bonzini  <bonzini@gnu.org>

        PR rtl-optimization/40741
        * config/arm/arm.c (thumb1_rtx_costs): IOR or XOR with
        a small constant is cheap.
        * config/arm/arm.md (andsi3, iorsi3): Try to place the result of
        force_reg on the LHS.
        (xorsi3): Likewise, and split the XOR if the constant is complex
        and not in Thumb mode.

  2009-10-28  Paolo Bonzini  <bonzini@gnu.org>

        PR rtl-optimization/40741
        * gcc.target/arm/thumb-branch1.c: New.
........
  r153651 | bonzini | 2009-10-28 04:27:15 -0600 (Wed, 28 Oct 2009) | 13 lines

  2009-10-28  Paolo Bonzini  <bonzini@gnu.org>

        PR rtl-optimization/39715
        * combine.c (simplify_comparison): Use extensions to
        widen comparisons.  Try an ANDing first.

  testsuite:
  2009-10-28  Paolo Bonzini  <bonzini@gnu.org>

        PR rtl-optimization/39715
        * gcc.target/arm/thumb-bitfld1.c: New.
........
  r153652 | bonzini | 2009-10-28 06:37:30 -0600 (Wed, 28 Oct 2009) | 13 lines

  2009-10-28  Paolo Bonzini  <bonzini@gnu.org>

        PR rtl-optimization/41812

        Revert:
        2009-06-27  Paolo Bonzini  <bonzini@gnu.org>

        * df-problems.c (df_md_scratch): New.
        (df_md_alloc, df_md_free): Allocate/free it.
        (df_md_local_compute): Only include live registers in init.
        (df_md_transfer_function): Prune the in-set computed by
        the confluence function, and the gen-set too.
........
  r153654 | paolo | 2009-10-28 07:07:00 -0600 (Wed, 28 Oct 2009) | 6 lines

  2009-10-28  Paolo Carlini  <paolo.carlini@oracle.com>

        * include/bits/stl_iterator_base_funcs.h: (next): Change
        template parameter name consistently with the resolution
        of DR 1011 ([Ready] in Santa Cruz).
........
  r153655 | rguenth | 2009-10-28 07:28:32 -0600 (Wed, 28 Oct 2009) | 14 lines

  2009-10-28  Richard Guenther  <rguenther@suse.de>

        PR middle-end/41855
        * tree-ssa-alias.c (refs_may_alias_p_1): Deal with CONST_DECLs
        (ref_maybe_used_by_call_p_1): Fix bcopy handling.
        (call_may_clobber_ref_p_1): Likewise.
        * tree-ssa-structalias.c (find_func_aliases): Likewise.
        * alias.c (nonoverlapping_memrefs_p): Deal with CONST_DECLs.

        * gfortran.dg/lto/20091028-1_0.f90: New testcase.
        * gfortran.dg/lto/20091028-1_1.c: Likewise.
        * gfortran.dg/lto/20091028-2_0.f90: Likewise.
        * gfortran.dg/lto/20091028-2_1.c: Likewise.
........
  r153656 | charlet | 2009-10-28 07:31:51 -0600 (Wed, 28 Oct 2009) | 25 lines

  2009-10-28  Robert Dewar  <dewar@adacore.com>

        * a-ztexio.adb, a-ztexio.ads, a-witeio.ads, a-witeio.adb,
        a-textio.ads, a-textio.adb: Reorganize (moving specs from private part
        to body).
        (Initialize_Standard_Files): New procedure.
        * a-tienau.adb: Minor change to make EOF directly visible
        * a-tirsfi.ads, a-wrstfi.adb, a-wrstfi.ads, a-zrstfi.adb,
        a-zrstfi.ads, a-tirsfi.adb: New unit, initial version.
        * gnat_rm.texi: Add documentation for
        Ada.[Wide_[Wide_]]Text_IO.Reset_Standard_Files.
        * Makefile.rtl: Add entries for
        Ada.[Wide_[Wide_]]Text_IO.Reset_Standard_Files

  2009-10-28  Thomas Quinot  <quinot@adacore.com>

        * exp_ch9.ads: Minor reformatting
        * sem_ch3.adb: Minor reformatting
        * sem_aggr.adb: Minor reformatting.
        * sem_attr.adb: Minor reformatting
        * tbuild.adb, tbuild.ads, par-ch4.adb, exp_ch4.adb
(Tbuild.New_Op_Node):
        New subprogram.
        Minor code reorganization/factoring.
........
  r153657 | charlet | 2009-10-28 07:41:05 -0600 (Wed, 28 Oct 2009) | 29 lines

  2009-10-28  Thomas Quinot  <quinot@adacore.com>

        * exp_ch4.adb (Expand_N_Type_Conversion): Perform Integer promotion for
        the operand of the unary minus and ABS operators.

        * sem_type.adb (Covers): A concurrent type and its corresponding record
        type are compatible.
        * exp_attr.adb (Expand_N_Attribute_Reference): Do not rewrite a 'Access
        attribute reference for the current instance of a protected type while
        analyzing an access discriminant constraint in a component definition.
        Such a reference is handled in the corresponding record's init proc,
        while initializing the constrained component.
        * exp_ch9.adb (Expand_N_Protected_Type_Declaration): When creating the
        corresponding record type, propagate components'
        Has_Per_Object_Constraint flag.
        * exp_ch3.adb (Build_Init_Procedure.Build_Init_Statements):
        For a concurrent type, set up concurrent aspects before initializing
        components with a per object constrain, because they may be controlled,
        and their initialization may call entries or protected subprograms of
        the enclosing concurrent object.

  2009-10-28  Emmanuel Briot  <briot@adacore.com>

        * prj-nmsc.adb (Add_If_Not_In_List): New subprogram, for better sharing
        of code.
        (Find_Source_Dirs): resolve links if Opt.Follow_Links_For_Dirs when
        processing the directories specified explicitly in the project file.
........
  r153658 | charlet | 2009-10-28 07:50:10 -0600 (Wed, 28 Oct 2009) | 10 lines

  2009-10-28  Robert Dewar  <dewar@adacore.com>

        * exp_attr.adb, exp_ch9.adb, prj-nmsc.adb, tbuild.adb, ali.adb,
        types.ads: Minor reformatting

  2009-10-28  Tristan Gingold  <gingold@adacore.com>

        * init.c: Fix __gnat_error_handler for Darwin10 (Snow Leopard)
........
  r153659 | rguenth | 2009-10-28 07:52:20 -0600 (Wed, 28 Oct 2009) | 11 lines

  2009-10-28  Richard Guenther  <rguenther@suse.de>

        * tree.c (free_lang_data_in_type): Do not call get_alias_set.
        (free_lang_data): Unconditionally compute alias sets for all
        standard integer types.  Bail out if gate bailed out previously.
        Do not reset the types_compatible_p langhook.
        (gate_free_lang_data): Remove.
        (struct pass_ipa_free_lang_data): Enable unconditionally.
        * gimple.c (gimple_get_alias_set): Use the same alias-set for
        all pointer types.
........
  r153660 | charlet | 2009-10-28 08:07:16 -0600 (Wed, 28 Oct 2009) | 2 lines

        * gcc-interface/Make-lang.in: Update dependencies.
........
  r153661 | charlet | 2009-10-28 08:09:12 -0600 (Wed, 28 Oct 2009) | 22 lines

  2009-10-28  Vincent Celier  <celier@adacore.com>

        * prj-nmsc.adb (Add_To_Or_Remove_From_List): New name of procedure
        Add_If_Not_In_List to account to the fact that a directory may be
        removed from the list. Only remove directory if Removed is True.

  2009-10-28  Gary Dismukes  <dismukes@adacore.com>

        * a-textio.ads, a-textio.ads: Put back function EOF_Char in private
        part. Put back body of function EOF_Char.
        * a-tienau.adb: Remove with of Interfaces.C_Streams and change EOF back
        to EOF_Char.

  2009-10-28  Emmanuel Briot  <briot@adacore.com>

        * prj-tree.adb (Free): Fix memory leak.

  2009-10-28  Thomas Quinot  <quinot@adacore.com>

        * s-fileio.adb: Minor reformatting
........
  r153662 | charlet | 2009-10-28 08:14:05 -0600 (Wed, 28 Oct 2009) | 9 lines

  2009-10-28  Thomas Quinot  <quinot@adacore.com>

        * s-crtl.ads (System.CRTL.strerror): New function.

  2009-10-28  Ed Schonberg  <schonberg@adacore.com>

        * sem_type.adb: Add guard to recover some type errors.
........
  r153663 | charlet | 2009-10-28 08:22:09 -0600 (Wed, 28 Oct 2009) | 12 lines

  2009-10-28  Bob Duff  <duff@adacore.com>

        * s-fileio.adb: Give more information in exception messages.

  2009-10-28  Robert Dewar  <dewar@adacore.com>

        * gnat_ugn.texi: Document new -gnatyt requirement for space after right
        paren if next token starts with digit or letter.
        * styleg.adb (Check_Right_Paren): New rule for space after if next
        character is a letter or digit.
........
  r153664 | rguenth | 2009-10-28 08:33:17 -0600 (Wed, 28 Oct 2009) | 4 lines

  2009-10-28  Richard Guenther  <rguenther@suse.de>

          * gimple.c (gimple_get_alias_set): Fix comment typo.
........
  r153665 | jakub | 2009-10-28 08:36:28 -0600 (Wed, 28 Oct 2009) | 3 lines

        * var-tracking.c (emit_note_insn_var_location): Get the mode of
        a variable part from its REG, MEM or VALUE.
........
  r153666 | jakub | 2009-10-28 08:37:24 -0600 (Wed, 28 Oct 2009) | 4 lines

        * var-tracking.c (emit_note_insn_var_location): Don't call the second
        vt_expand_loc unnecessarily when location is not a register nor
        memory.
........
  r153667 | jakub | 2009-10-28 08:39:06 -0600 (Wed, 28 Oct 2009) | 6 lines

        PR target/41762
        * config/i386/i386.c (ix86_pic_register_p): Don't call
        rtx_equal_for_cselib_p for VALUEs discarded as useless.

        * gcc.dg/pr41762.c: New test.
........
  r153669 | jakub | 2009-10-28 08:43:04 -0600 (Wed, 28 Oct 2009) | 6 lines

        PR debug/41801
        * builtins.c (get_builtin_sync_mem): Expand loc in ptr_mode,
        call convert_memory_address on addr.

        * g++.dg/ext/sync-3.C: New test.
........
  r153670 | jakub | 2009-10-28 08:45:03 -0600 (Wed, 28 Oct 2009) | 6 lines

        PR middle-end/41837
        * ipa-struct-reorg.c (find_field_in_struct_1): Return NULL if
        fields don't have DECL_NAME.

        * gcc.dg/pr41837.c: New test.
........
  r153671 | rguenth | 2009-10-28 08:48:34 -0600 (Wed, 28 Oct 2009) | 15 lines

  2009-10-28  Richard Guenther  <rguenther@suse.de>

        PR lto/41808
        PR lto/41839
        * tree-ssa.c (useless_type_conversion_p): Do not treat
        conversions to pointers to incomplete types as useless.
        * gimple.c (gimple_types_compatible_p): Compare struct tags,
        not typedef names.

        * gcc.dg/lto/20091027-1_0.c: New testcase.
        * gcc.dg/lto/20091027-1_1.c: Likewise.
        * g++.dg/lto/20091026-1_0.C: Likewise.
        * g++.dg/lto/20091026-1_1.C: Likewise.
        * g++.dg/lto/20091026-1_a.h: Likewise.
........

Modified:
    branches/reload-v2a/   (props changed)

Propchange: branches/reload-v2a/
            ('svnmerge-integrated' modified)


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

* [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend of a builtin function
  2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
                   ` (15 preceding siblings ...)
  2009-10-29 16:53 ` law at gcc dot gnu dot org
@ 2009-10-30 16:07 ` hjl at gcc dot gnu dot org
  16 siblings, 0 replies; 19+ messages in thread
From: hjl at gcc dot gnu dot org @ 2009-10-30 16:07 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #14 from hjl at gcc dot gnu dot org  2009-10-30 16:05 -------
Subject: Bug 41020

Author: hjl
Date: Fri Oct 30 16:04:41 2009
New Revision: 153759

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=153759
Log:
2009-10-30  H.J. Lu  <hongjiu.lu@intel.com>

        Backport from mainline:
        2009-10-30  Dodji Seketeli  <dodji@redhat.com>

        PR c++/41863
        * g++.dg/template/sizeof12.C: New test.

        2009-10-29  Martin Jambor  <mjambor@suse.cz>

        PR tree-optimization/41775
        * g++.dg/torture/pr41775.C: New testcase.

        2009-10-28  Jakub Jelinek  <jakub@redhat.com>

        PR debug/41801
        * g++.dg/ext/sync-3.C: New test.

        2009-10-27  Jakub Jelinek  <jakub@redhat.com>

        PR c++/41020
        * g++.dg/lookup/extern-c-redecl5.C: Fix up regexp.

        2009-10-26  Jakub Jelinek  <jakub@redhat.com>

        PR bootstrap/41345
        * gcc.dg/pr41345.c: New test.

        2009-10-26  Dodji Seketeli  <dodji@redhat.com>

        PR c++/41785
        * g++.dg/cpp0x/variadic96.C: New test.

        2009-10-26  Dodji Seketeli  <dodji@redhat.com>

        PR c++/41020
        * g++.dg/lookup/extern-c-redecl2.C: New test.
        * g++.dg/lookup/extern-c-redecl3.C: Likewise.
        * g++.dg/lookup/extern-c-redecl4.C: Likewise.
        * g++.dg/lookup/extern-c-redecl5.C: Likewise.

        2009-10-23  Joseph Myers  <joseph@codesourcery.com>

        PR c/40033
        * gcc.dg/noncompile/pr40033-1.c: New test.

        2009-10-23  Joseph Myers  <joseph@codesourcery.com>

        PR c/41673
        * gcc.dg/Wstrict-aliasing-bogus-vla-1.c: New test.

        2009-10-21  Sebastian Pop  <sebastian.pop@amd.com>

        PR tree-optimization/41497
        * gcc.dg/tree-ssa/pr41497.c: New.

Added:
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/cpp0x/variadic96.C
      - copied unchanged from r153757,
trunk/gcc/testsuite/g++.dg/cpp0x/variadic96.C
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/ext/sync-3.C
      - copied unchanged from r153757, trunk/gcc/testsuite/g++.dg/ext/sync-3.C
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/lookup/extern-c-redecl2.C
      - copied unchanged from r153757,
trunk/gcc/testsuite/g++.dg/lookup/extern-c-redecl2.C
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/lookup/extern-c-redecl3.C
      - copied unchanged from r153757,
trunk/gcc/testsuite/g++.dg/lookup/extern-c-redecl3.C
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/lookup/extern-c-redecl4.C
      - copied unchanged from r153757,
trunk/gcc/testsuite/g++.dg/lookup/extern-c-redecl4.C
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/lookup/extern-c-redecl5.C
      - copied unchanged from r153757,
trunk/gcc/testsuite/g++.dg/lookup/extern-c-redecl5.C
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/template/sizeof12.C
      - copied unchanged from r153757,
trunk/gcc/testsuite/g++.dg/template/sizeof12.C
    branches/gcc-4_4-branch/gcc/testsuite/g++.dg/torture/pr41775.C
      - copied unchanged from r153757,
trunk/gcc/testsuite/g++.dg/torture/pr41775.C
    branches/gcc-4_4-branch/gcc/testsuite/gcc.dg/Wstrict-aliasing-bogus-vla-1.c
      - copied unchanged from r153758,
trunk/gcc/testsuite/gcc.dg/Wstrict-aliasing-bogus-vla-1.c
    branches/gcc-4_4-branch/gcc/testsuite/gcc.dg/noncompile/pr40033-1.c
      - copied unchanged from r153758,
trunk/gcc/testsuite/gcc.dg/noncompile/pr40033-1.c
    branches/gcc-4_4-branch/gcc/testsuite/gcc.dg/pr41345.c
      - copied unchanged from r153757, trunk/gcc/testsuite/gcc.dg/pr41345.c
    branches/gcc-4_4-branch/gcc/testsuite/gcc.dg/tree-ssa/pr41497.c
      - copied unchanged from r153758,
trunk/gcc/testsuite/gcc.dg/tree-ssa/pr41497.c
Modified:
    branches/gcc-4_4-branch/gcc/testsuite/ChangeLog


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020


^ permalink raw reply	[flat|nested] 19+ messages in thread

end of thread, other threads:[~2009-10-30 16:07 UTC | newest]

Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-08-09 21:30 [Bug c++/41020] New: Can't declare an extern "C" friend davek at gcc dot gnu dot org
2009-08-09 21:32 ` [Bug c++/41020] " davek at gcc dot gnu dot org
2009-08-10 10:40 ` jwakely dot gcc at gmail dot com
2009-08-10 12:45 ` [Bug c++/41020] [4.5 Regression] " rguenth at gcc dot gnu dot org
2009-08-10 16:17 ` davek at gcc dot gnu dot org
2009-08-10 17:05 ` jwakely dot gcc at gmail dot com
2009-08-10 17:16 ` davek at gcc dot gnu dot org
2009-08-24  1:00 ` mmitchel at gcc dot gnu dot org
2009-09-18  3:56 ` pinskia at gcc dot gnu dot org
2009-10-04 19:14 ` [Bug c++/41020] [4.5 Regression] Can't declare an extern "C" friend of a builtin function pinskia at gcc dot gnu dot org
2009-10-23 18:19   ` Dodji Seketeli
2009-10-23 18:20 ` dodji at redhat dot com
2009-10-23 18:34 ` dodji at gcc dot gnu dot org
2009-10-23 19:32 ` dodji at redhat dot com
2009-10-26 14:40 ` dodji at gcc dot gnu dot org
2009-10-26 16:06 ` dodji at gcc dot gnu dot org
2009-10-27 10:29 ` jakub at gcc dot gnu dot org
2009-10-29 16:53 ` law at gcc dot gnu dot org
2009-10-30 16:07 ` hjl at gcc dot gnu dot org

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).