Tell trans that user unops are by value

Jorge Aparicio · Jorge Aparicio · commit 9a962a7bdc9a · 2014-12-18T14:56:00.000-05:00
diff --git a/src/librustc_trans/trans/expr.rs b/src/librustc_trans/trans/expr.rs
@@ -1101,11 +1101,11 @@ fn trans_rvalue_dps_unadjusted<'blk, 'tcx>(bcx: Block<'blk, 'tcx>,
                                 vec![(rhs_datum, rhs.id)], Some(dest),
                                 !ast_util::is_by_value_binop(op)).bcx
         }
-        ast::ExprUnary(_, ref subexpr) => {
+        ast::ExprUnary(op, ref subexpr) => {
             // if not overloaded, would be RvalueDatumExpr
             let arg = unpack_datum!(bcx, trans(bcx, &**subexpr));
             trans_overloaded_op(bcx, expr, MethodCall::expr(expr.id),
-                                arg, Vec::new(), Some(dest), true).bcx
+                                arg, Vec::new(), Some(dest), !ast_util::is_by_value_unop(op)).bcx
         }
         ast::ExprIndex(ref base, ref idx) => {
             // if not overloaded, would be RvalueDatumExpr

Original file line number	Diff line number	Diff line change
`@@ -1101,11 +1101,11 @@ fn trans_rvalue_dps_unadjusted<'blk, 'tcx>(bcx: Block<'blk, 'tcx>,`
`1101`	`1101`	`vec![(rhs_datum, rhs.id)], Some(dest),`
`1102`	`1102`	`!ast_util::is_by_value_binop(op)).bcx`
`1103`	`1103`	`}`
`1104`		`- ast::ExprUnary(_, ref subexpr) => {`
	`1104`	`+ ast::ExprUnary(op, ref subexpr) => {`
`1105`	`1105`	`// if not overloaded, would be RvalueDatumExpr`
`1106`	`1106`	`let arg = unpack_datum!(bcx, trans(bcx, &**subexpr));`
`1107`	`1107`	`trans_overloaded_op(bcx, expr, MethodCall::expr(expr.id),`
`1108`		`- arg, Vec::new(), Some(dest), true).bcx`
	`1108`	`+ arg, Vec::new(), Some(dest), !ast_util::is_by_value_unop(op)).bcx`
`1109`	`1109`	`}`
`1110`	`1110`	`ast::ExprIndex(ref base, ref idx) => {`
`1111`	`1111`	`// if not overloaded, would be RvalueDatumExpr`