diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index f60735d61659..9665be4dfc71 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,3 +1,10 @@ +2013-02-07 Jason Merrill + + PR c++/56235 + * method.c (do_build_copy_constructor): Don't bother turning + scalars from lvalues to xvalues. + (do_build_copy_assign): Likewise. + 2013-02-06 Jason Merrill * parser.c (cp_parser_enum_specifier): Check for error_mark_node. diff --git a/gcc/cp/method.c b/gcc/cp/method.c index d13a0cf3a9ee..a1bab95049ca 100644 --- a/gcc/cp/method.c +++ b/gcc/cp/method.c @@ -610,7 +610,9 @@ do_build_copy_constructor (tree fndecl) } init = build3 (COMPONENT_REF, expr_type, parm, field, NULL_TREE); - if (move_p && TREE_CODE (expr_type) != REFERENCE_TYPE) + if (move_p && TREE_CODE (expr_type) != REFERENCE_TYPE + /* 'move' breaks bit-fields, and has no effect for scalars. */ + && !scalarish_type_p (expr_type)) init = move (init); init = build_tree_list (NULL_TREE, init); @@ -724,7 +726,9 @@ do_build_copy_assign (tree fndecl) expr_type = cp_build_qualified_type (expr_type, quals); init = build3 (COMPONENT_REF, expr_type, init, field, NULL_TREE); - if (move_p && TREE_CODE (expr_type) != REFERENCE_TYPE) + if (move_p && TREE_CODE (expr_type) != REFERENCE_TYPE + /* 'move' breaks bit-fields, and has no effect for scalars. */ + && !scalarish_type_p (expr_type)) init = move (init); if (DECL_NAME (field)) diff --git a/gcc/testsuite/g++.dg/init/bitfield4.C b/gcc/testsuite/g++.dg/init/bitfield4.C new file mode 100644 index 000000000000..30041c435999 --- /dev/null +++ b/gcc/testsuite/g++.dg/init/bitfield4.C @@ -0,0 +1,24 @@ +// PR c++/56235 + +struct A +{ + A (const A &); +}; + +struct B +{ + A a; + enum Mode { }; + Mode m:8; +}; + +struct C +{ + C(); + B b; +}; + +C fn() +{ + return C(); +}