[Bug c++/32756] New: wrong ambiguous overload error?

[Bug c++/32756] New: wrong ambiguous overload error?

[mueller at gcc dot gnu dot org]

Hi, 

this might be invalid, needs verification. starting with gcc 4.3, the following
testcase is rejected: 

=== Cut ===

class QString;

struct QByteArray
{
  QByteArray ();
  bool operator!= (const QString & s2) const;
};

bool operator!= (const QByteArray & a1, const QByteArray & a2);

struct QString
{
  QString ();
  QString (const QByteArray & a);
};

QByteArray abbreviation ();

void
fromString ()
{
  QByteArray zoneAbbrev;
  if (abbreviation () != zoneAbbrev)
    {
    }
}
=== Cut ===
ambiguity.cc:23: error: ISO C++ says that these are ambiguous, even though the
worst conversion for the first is better than the worst conversion for the
second:
ambiguity.cc:9: note: candidate 1: bool operator!=(const QByteArray&, const
QByteArray&)
ambiguity.cc:6: note: candidate 2: bool QByteArray::operator!=(const QString&)
const


-- 
           Summary: wrong ambiguous overload error?
           Product: gcc
           Version: 4.3.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: mueller at gcc dot gnu dot org


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

[Bug c++/32756] [4.3 Regression] wrong ambiguous overload error?

[mueller at gcc dot gnu dot org]

------- Comment #1 from mueller at gcc dot gnu dot org  2007-07-14 07:33 -------
this was introduced by the following patch:

2007-05-30  Russell Yanofsky <russ@yanofsky.org>
            Douglas Gregor <doug.gregor@gmail.com>
            Pedro Lamarao <pedro.lamarao@mndfck.org>
            Howard Hinnant <howard.hinnant@gmail.com>

        PR c++/7412
        PR c++/29939


-- 

mueller at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |doug dot gregor at gmail dot
                   |                            |com


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

[Bug c++/32756] [4.3 Regression] wrong ambiguous overload error?

[pinskia at gcc dot gnu dot org]

-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pinskia at gcc dot gnu dot
                   |                            |org
   Target Milestone|---                         |4.3.0


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

[Bug c++/32756] [4.3 Regression] wrong ambiguous overload error?

[mueller at gcc dot gnu dot org]

------- Comment #2 from mueller at gcc dot gnu dot org  2007-07-14 10:00 -------
Created an attachment (id=13914)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=13914&action=view)
patch

the following patch moves the change of ranking described in the working draft
under a "dialect" check, which hot-fixes this bug


-- 


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

[Bug c++/32756] [4.3 Regression] wrong ambiguous overload error?

[nathan at gcc dot gnu dot org]

------- Comment #3 from nathan at gcc dot gnu dot org  2007-08-07 09:32 -------
This is something to do with the rvalue reference patches


-- 

nathan at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |nathan at gcc dot gnu dot
                   |dot org                     |org
             Status|UNCONFIRMED                 |ASSIGNED
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2007-08-07 09:32:12
               date|                            |


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

[Bug c++/32756] [4.3 Regression] wrong ambiguous overload error?

[mmitchel at gcc dot gnu dot org]

-- 

mmitchel at gcc dot gnu dot org changed:

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


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

[Bug c++/32756] [4.3 Regression] wrong ambiguous overload error?

[mueller at gcc dot gnu dot org]

------- Comment #4 from mueller at gcc dot gnu dot org  2007-08-20 11:13 -------
ping.. any results?


-- 


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

[Bug c++/32756] [4.3 Regression] wrong ambiguous overload error?

[nathan at gcc dot gnu dot org]

------- Comment #5 from nathan at gcc dot gnu dot org  2007-08-20 12:21 -------
I have a patch, which needs fuller testing.  (I've been on vacation :))


-- 


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

[Bug c++/32756] [4.3 Regression] wrong ambiguous overload error?

[mueller at gcc dot gnu dot org]

------- Comment #6 from mueller at gcc dot gnu dot org  2007-08-20 12:43 -------
I`d be happy to help with testing :)


-- 


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

[Bug c++/32756] [4.3 Regression] wrong ambiguous overload error?

[nathan at gcc dot gnu dot org]

------- Comment #7 from nathan at gcc dot gnu dot org  2007-09-16 17:26 -------
Subject: Bug 32756

Author: nathan
Date: Sun Sep 16 17:26:42 2007
New Revision: 128528

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=128528
Log:
        cp/
        PR c++/32756
        * call.c (maybe_handle_implicit_object): Set this_p, clear
        rvaluedness_matches_p.
        (compare_ics): Do not compare rvaluedness matching when one of the
        operands is an implicit object.

        testsuite/
        PR c++/32756
        * g++.dg/overload/operator3.C: New.

Added:
    trunk/gcc/testsuite/g++.dg/overload/operator3.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/call.c
    trunk/gcc/testsuite/ChangeLog


-- 


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

[Bug c++/32756] [4.3 Regression] wrong ambiguous overload error?

[bangerth at dealii dot org]

------- Comment #8 from bangerth at dealii dot org  2007-09-18 19:13 -------
This works with a snapshot from yesterday.


-- 

bangerth at dealii dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bangerth at dealii dot org
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


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