[Bug c++/56235] New: [4.8 regression] Bogus "error: invalid conversion from ‘unsigned char’ to ‘B::Mode’ [-fpermissive]"

[Bug c++/56235] New: [4.8 regression] Bogus "error: invalid conversion from ‘unsigned char’ to ‘B::Mode’ [-fpermissive]"

[ppluzhnikov at google dot com]

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

             Bug #: 56235
           Summary: [4.8 regression] Bogus "error: invalid conversion from
                    ‘unsigned char’ to ‘B::Mode’ [-fpermissive]"
    Classification: Unclassified
           Product: gcc
           Version: unknown
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: ppluzhnikov@google.com


Test case:

struct A
{
    A (const A &);
};

struct B
{
    A a;
    enum Mode { };
    Mode m:8;
};

struct C
{
    C();
    B b;
};

C fn()
{
    return C();
}


The test compiles fine with 4.7, fails with 4.8 (20130205 (experimental)) but
only in C++11 mode:

g++ -c -std=c++11 t.ii
t.ii: In constructor ‘B::B(B&&)’:
t.ii:6:8: error: invalid conversion from ‘unsigned char’ to ‘B::Mode’
[-fpermissive]
 struct B
        ^
t.ii: In constructor ‘C::C(C&&)’:
t.ii:13:8: note: synthesized method ‘B::B(B&&)’ first required here 
 struct C
        ^
t.ii: In function ‘C fn()’:
t.ii:21:14: note: synthesized method ‘C::C(C&&)’ first required here 
     return C();
              ^


Google ref: b/8152830

[Bug c++/56235] [4.8 regression] Bogus "error: invalid conversion from ‘unsigned char’ to ‘B::Mode’ [-fpermissive]"

[jakub at gcc dot gnu.org]

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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2013-02-07
                 CC|                            |jakub at gcc dot gnu.org,
                   |                            |jason at gcc dot gnu.org
   Target Milestone|---                         |4.8.0
     Ever Confirmed|0                           |1

--- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> 2013-02-07 08:59:54 UTC ---
Started with http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=192813

[Bug c++/56235] [4.8 regression] Bogus "error: invalid conversion from ‘unsigned char’ to ‘B::Mode’ [-fpermissive]"

[jason at gcc dot gnu.org]

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

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|unassigned at gcc dot       |jason at gcc dot gnu.org
                   |gnu.org                     |

[Bug c++/56235] [4.8 regression] Bogus "error: invalid conversion from ‘unsigned char’ to ‘B::Mode’ [-fpermissive]"

[jason at gcc dot gnu.org]

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

--- Comment #2 from Jason Merrill <jason at gcc dot gnu.org> 2013-02-07 16:32:39 UTC ---
Author: jason
Date: Thu Feb  7 16:32:28 2013
New Revision: 195854

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=195854
Log:
    PR c++/56235
    * method.c (do_build_copy_constructor): Don't bother turning
    scalars from lvalues to xvalues.
    (do_build_copy_assign): Likewise.

Added:
    trunk/gcc/testsuite/g++.dg/init/bitfield4.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/method.c

[Bug c++/56235] [4.8 regression] Bogus "error: invalid conversion from ‘unsigned char’ to ‘B::Mode’ [-fpermissive]"

[jason at gcc dot gnu.org]

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

Jason Merrill <jason at gcc dot gnu.org> changed:

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

--- Comment #3 from Jason Merrill <jason at gcc dot gnu.org> 2013-02-07 17:15:06 UTC ---
Fixed.