---

yaml
---
r: 56279
b: refs/heads/auto
c: f93b3cd
h: refs/heads/master
i:
  56277: 6e5f1f9
  56275: 5bc8e4d
  56271: 7c2a389
v: v3

Author: pcwalton

[refs]

@@ -14,6 +14,6 @@ refs/heads/try3: 9387340aab40a73e8424c48fd42f0c521a4875c0
 refs/tags/release-0.3.1: 495bae036dfe5ec6ceafd3312b4dca48741e845b
 refs/tags/release-0.4: e828ea2080499553b97dfe33b3f4d472b4562ad7
 refs/tags/release-0.5: 7e3bcfbf21278251ee936ad53e92e9b719702d73
-refs/heads/auto: af4ea11d09618b800ebec477333a373a75a46d96
+refs/heads/auto: f93b3cd5c3783eabd527607adda891d8a84dab4f
 refs/heads/servo: af82457af293e2a842ba6b7759b70288da276167
 refs/tags/release-0.6: b4ebcfa1812664df5e142f0134a5faea3918544c

branches/auto/src/librustc/middle/trans/cabi.rs

@@ -13,7 +13,6 @@ use middle::trans::base::*;
 use middle::trans::build::*;
 use middle::trans::common::*;
 
-use core::io::println;
 use core::libc::c_uint;
 use core::option;
 use core::vec;

branches/auto/src/librustc/middle/trans/foreign.rs

@@ -185,17 +185,19 @@ fn build_wrap_fn_(ccx: @CrateContext,
                   llshimfn: ValueRef,
                   llwrapfn: ValueRef,
                   shim_upcall: ValueRef,
+                  needs_c_return: bool,
                   arg_builder: wrap_arg_builder,
                   ret_builder: wrap_ret_builder) {
     let _icx = ccx.insn_ctxt("foreign::build_wrap_fn_");
     let fcx = new_fn_ctxt(ccx, ~[], llwrapfn, tys.fn_sig.output, None);
 
-    // Patch up the return type if it's not immediate.
-    /*if !ty::type_is_immediate(tys.fn_sig.output) {
+    // Patch up the return type if it's not immediate and we're returning via
+    // the C ABI.
+    if needs_c_return && !ty::type_is_immediate(tys.fn_sig.output) {
         let lloutputtype = type_of::type_of(*fcx.ccx, tys.fn_sig.output);
         fcx.llretptr = Some(alloca(raw_block(fcx, false, fcx.llstaticallocas),
                                    lloutputtype));
-    }*/
+    }
 
     let bcx = top_scope_block(fcx, None);
     let lltop = bcx.llbb;
@@ -499,6 +501,7 @@ pub fn trans_foreign_mod(ccx: @CrateContext,
                        llshimfn,
                        llwrapfn,
                        ccx.upcalls.call_shim_on_c_stack,
+                       false,
                        build_args,
                        build_ret);
 
@@ -1229,6 +1232,7 @@ pub fn trans_foreign_fn(ccx: @CrateContext,
                        llshimfn,
                        llwrapfn,
                        ccx.upcalls.call_shim_on_rust_stack,
+                       true,
                        build_args,
                        build_ret);
 

branches/auto/src/test/bench/shootout-reverse-complement.rs

@@ -1,3 +1,5 @@
+// xfail-pretty
+
 use core::cast::transmute;
 use core::libc::{STDOUT_FILENO, c_int, fdopen, fgets, fopen, fputc, fwrite};
 use core::libc::{size_t};