Don't mangle probes all the time

This commit is contained in:
Alex Crichton 2017-07-07 11:20:04 -07:00
parent f9f6bd0589
commit 91eaa85902
6 changed files with 26 additions and 15 deletions

View File

@ -21,9 +21,14 @@ fn main() {
#[cfg(feature = "gen-tests")] #[cfg(feature = "gen-tests")]
tests::generate(); tests::generate();
// Build missing intrinsics from compiler-rt C source code // Build missing intrinsics from compiler-rt C source code. If we're
// mangling names though we assume that we're also in test mode so we don't
// build anything and we rely on the upstream implementation of compiler-rt
// functions
if !cfg!(feature = "mangled-names") {
#[cfg(feature = "c")] #[cfg(feature = "c")]
c::compile(&llvm_target); c::compile(&llvm_target);
}
// To compile intrinsics.rs for thumb targets, where there is no libc // To compile intrinsics.rs for thumb targets, where there is no libc
if llvm_target[0].starts_with("thumb") { if llvm_target[0].starts_with("thumb") {
@ -4099,11 +4104,9 @@ mod c {
// also needs to satisfy intrinsics that jemalloc or C in general may // also needs to satisfy intrinsics that jemalloc or C in general may
// need, so include a few more that aren't typically needed by // need, so include a few more that aren't typically needed by
// LLVM/Rust. // LLVM/Rust.
if env::var_os("CARGO_FEATURE_RUSTBUILD").is_some() {
sources.extend(&[ sources.extend(&[
"ffsdi2.c", "ffsdi2.c",
]); ]);
}
if target_os != "ios" { if target_os != "ios" {
sources.extend( sources.extend(

View File

@ -93,6 +93,7 @@ for rlib in $(echo $path); do
uniq -d | \ uniq -d | \
grep -v __x86.get_pc_thunk | \ grep -v __x86.get_pc_thunk | \
grep -v __builtin_cl | \ grep -v __builtin_cl | \
grep -v __builtin_ctz | \
grep 'T __' grep 'T __'
if test $? = 0; then if test $? = 0; then

View File

@ -13,6 +13,8 @@
#![feature(lang_items)] #![feature(lang_items)]
#![feature(start)] #![feature(start)]
#![feature(i128_type)] #![feature(i128_type)]
#![feature(global_allocator)]
#![feature(allocator_api)]
#![cfg_attr(windows, feature(panic_unwind))] #![cfg_attr(windows, feature(panic_unwind))]
#![no_std] #![no_std]
@ -22,6 +24,10 @@ extern crate compiler_builtins;
#[cfg(windows)] #[cfg(windows)]
extern crate panic_unwind; extern crate panic_unwind;
#[cfg(not(thumb))]
#[global_allocator]
static A: alloc_system::System = alloc_system::System;
// NOTE cfg(not(thumbv6m)) means that the operation is not supported on ARMv6-M at all. Not even // NOTE cfg(not(thumbv6m)) means that the operation is not supported on ARMv6-M at all. Not even
// compiler-rt provides a C/assembly implementation. // compiler-rt provides a C/assembly implementation.

View File

@ -112,8 +112,9 @@ intrinsics! {
int_to_float!(i, u32, f64) int_to_float!(i, u32, f64)
} }
#[use_c_shim_if(all(any(target_arch = "x86", target_arch = "x86_64"), #[use_c_shim_if(all(not(target_env = "msvc"),
not(windows)))] any(target_arch = "x86",
all(not(windows), target_arch = "x86_64"))))]
#[arm_aeabi_alias = __aeabi_ul2d] #[arm_aeabi_alias = __aeabi_ul2d]
pub extern "C" fn __floatundidf(i: u64) -> f64 { pub extern "C" fn __floatundidf(i: u64) -> f64 {
int_to_float!(i, u64, f64) int_to_float!(i, u64, f64)

View File

@ -44,7 +44,7 @@
#![cfg(not(windows))] // Windows already has builtins to do this #![cfg(not(windows))] // Windows already has builtins to do this
#[naked] #[naked]
#[no_mangle] #[cfg_attr(not(feature = "mangled-names"), no_mangle)]
#[cfg(target_arch = "x86_64")] #[cfg(target_arch = "x86_64")]
pub unsafe extern fn __rust_probestack() { pub unsafe extern fn __rust_probestack() {
// Our goal here is to touch each page between %rsp+8 and %rsp+8-%rax, // Our goal here is to touch each page between %rsp+8 and %rsp+8-%rax,
@ -87,7 +87,7 @@ pub unsafe extern fn __rust_probestack() {
} }
#[naked] #[naked]
#[no_mangle] #[cfg_attr(not(feature = "mangled-names"), no_mangle)]
#[cfg(target_arch = "x86")] #[cfg(target_arch = "x86")]
pub unsafe extern fn __rust_probestack() { pub unsafe extern fn __rust_probestack() {
// This is the same as x86_64 above, only translated for 32-bit sizes. Note // This is the same as x86_64 above, only translated for 32-bit sizes. Note

View File

@ -10,7 +10,7 @@ use core::intrinsics;
#[cfg(windows)] #[cfg(windows)]
#[naked] #[naked]
#[no_mangle] #[cfg_attr(not(feature = "mangled-names"), no_mangle)]
pub unsafe fn ___chkstk_ms() { pub unsafe fn ___chkstk_ms() {
asm!("push %rcx asm!("push %rcx
push %rax push %rax
@ -34,7 +34,7 @@ pub unsafe fn ___chkstk_ms() {
#[cfg(windows)] #[cfg(windows)]
#[naked] #[naked]
#[no_mangle] #[cfg_attr(not(feature = "mangled-names"), no_mangle)]
pub unsafe fn __alloca() { pub unsafe fn __alloca() {
asm!("mov %rcx,%rax // x64 _alloca is a normal function with parameter in rcx asm!("mov %rcx,%rax // x64 _alloca is a normal function with parameter in rcx
jmp ___chkstk // Jump to ___chkstk since fallthrough may be unreliable"); jmp ___chkstk // Jump to ___chkstk since fallthrough may be unreliable");
@ -43,7 +43,7 @@ pub unsafe fn __alloca() {
#[cfg(windows)] #[cfg(windows)]
#[naked] #[naked]
#[no_mangle] #[cfg_attr(not(feature = "mangled-names"), no_mangle)]
pub unsafe fn ___chkstk() { pub unsafe fn ___chkstk() {
asm!("push %rcx asm!("push %rcx
cmp $$0x1000,%rax cmp $$0x1000,%rax