Skip to content

Commit 638ab87

Browse files
committed
---
yaml --- r: 225091 b: refs/heads/stable c: 74f4f39 h: refs/heads/master i: 225089: 0d1870b 225087: 8c5d908 v: v3
1 parent 221d3d5 commit 638ab87

File tree

2 files changed

+25
-10
lines changed

2 files changed

+25
-10
lines changed

[refs]

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,6 @@ refs/heads/tmp: e5d90d98402475b6e154ce216f9efcb80da1a747
2929
refs/tags/1.0.0-alpha.2: 4c705f6bc559886632d3871b04f58aab093bfa2f
3030
refs/tags/homu-tmp: 1fe32ca12c51afcd761d9962f51a74ff0d07a591
3131
refs/tags/1.0.0-beta: 8cbb92b53468ee2b0c2d3eeb8567005953d40828
32-
refs/heads/stable: 3d32cf5d9afdd8c375935ac875b2a20f24159e52
32+
refs/heads/stable: 74f4f395aefd1054b8d8974a128007d0671e9d78
3333
refs/tags/1.0.0: 55bd4f8ff2b323f317ae89e254ce87162d52a375
3434
refs/tags/1.1.0: bc3c16f09287e5545c1d3f76b7abd54f2eca868b

branches/stable/src/libstd/rt/unwind/mod.rs

Lines changed: 24 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -127,16 +127,31 @@ extern {}
127127
/// run.
128128
pub unsafe fn try<F: FnOnce()>(f: F) -> Result<(), Box<Any + Send>> {
129129
let mut f = Some(f);
130+
return inner_try(try_fn::<F>, &mut f as *mut _ as *mut c_void);
130131

131-
let prev = PANICKING.with(|s| s.get());
132-
PANICKING.with(|s| s.set(false));
133-
let ep = rust_try(try_fn::<F>, &mut f as *mut _ as *mut c_void);
134-
PANICKING.with(|s| s.set(prev));
135-
return if ep.is_null() {
136-
Ok(())
137-
} else {
138-
Err(imp::cleanup(ep))
139-
};
132+
// If an inner function were not used here, then this generic function `try`
133+
// uses the native symbol `rust_try`, for which the code is statically
134+
// linked into the standard library. This means that the DLL for the
135+
// standard library must have `rust_try` as an exposed symbol that
136+
// downstream crates can link against (because monomorphizations of `try` in
137+
// downstream crates will have a reference to the `rust_try` symbol).
138+
//
139+
// On MSVC this requires the symbol `rust_try` to be tagged with
140+
// `dllexport`, but it's easier to not have conditional `src/rt/rust_try.ll`
141+
// files and instead just have this non-generic shim the compiler can take
142+
// care of exposing correctly.
143+
unsafe fn inner_try(f: extern fn(*mut c_void), data: *mut c_void)
144+
-> Result<(), Box<Any + Send>> {
145+
let prev = PANICKING.with(|s| s.get());
146+
PANICKING.with(|s| s.set(false));
147+
let ep = rust_try(f, data);
148+
PANICKING.with(|s| s.set(prev));
149+
if ep.is_null() {
150+
Ok(())
151+
} else {
152+
Err(imp::cleanup(ep))
153+
}
154+
}
140155

141156
extern fn try_fn<F: FnOnce()>(opt_closure: *mut c_void) {
142157
let opt_closure = opt_closure as *mut Option<F>;

0 commit comments

Comments
 (0)