---

yaml
---
r: 34398
b: refs/heads/snap-stage3
c: 7b245d4
h: refs/heads/master
v: v3

Author: pcwalton

[refs]

@@ -1,7 +1,7 @@
 ---
 refs/heads/master: cd6f24f9d14ac90d167386a56e7a6ac1f0318195
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
-refs/heads/snap-stage3: 9abcacc0f371206a66cd1b346f5e46ac6063d9bd
+refs/heads/snap-stage3: 7b245d46ed5fc93d3251e9e07f50e9b09b1f42c4
 refs/heads/try: d324a424d8f84b1eb049b12cf34182bda91b0024
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
 refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b

branches/snap-stage3/src/librustc/middle/trans/common.rs

@@ -1094,6 +1094,8 @@ fn C_cstr(cx: @crate_ctxt, s: ~str) -> ValueRef {
     return g;
 }
 
+// NB: Do not use `do_spill_noroot` to make this into a constant string, or
+// you will be kicked off fast isel. See issue #4352 for an example of this.
 fn C_estr_slice(cx: @crate_ctxt, s: ~str) -> ValueRef {
     let cs = llvm::LLVMConstPointerCast(C_cstr(cx, s), T_ptr(T_i8()));
     C_struct(~[cs, C_uint(cx, str::len(s) + 1u /* +1 for null */)])

branches/snap-stage3/src/librustc/middle/trans/reflect.rs

@@ -46,8 +46,17 @@ impl reflector {
     }
 
     fn c_slice(s: ~str) -> ValueRef {
-        let ss = C_estr_slice(self.bcx.ccx(), s);
-        do_spill_noroot(self.bcx, ss)
+        // We're careful to not use first class aggregates here because that
+        // will kick us off fast isel. (Issue #4352.)
+        let bcx = self.bcx;
+        let str_vstore = ty::vstore_slice(ty::re_static);
+        let str_ty = ty::mk_estr(bcx.tcx(), str_vstore);
+        let scratch = scratch_datum(bcx, str_ty, false);
+        let c_str = PointerCast(bcx, C_cstr(bcx.ccx(), s), T_ptr(T_i8()));
+        Store(bcx, c_str, GEPi(bcx, scratch.val, [ 0, 0 ]));
+        let len = C_uint(bcx.ccx(), s.len() + 1);
+        Store(bcx, len, GEPi(bcx, scratch.val, [ 0, 1 ]));
+        scratch.val
     }
 
     fn c_size_and_align(t: ty::t) -> ~[ValueRef] {