Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Fix fast path of float parsing on x87
The fast path of the float parser relies on the rounding to happen
exactly and directly to the correct number of bits. On x87, instead,
double rounding would occour as the FPU stack defaults to 80 bits of
precision.

This can be fixed by setting the precision of the FPU stack before
performing the int to float conversion. This can be achieved by
changing the value of the x87 control word. This is a somewhat common
operation that is in fact performed whenever a float needs to be
truncated to an integer, but it is undesirable to add its overhead for
code that does not rely on x87 for computations (i.e. on non-x86
architectures, or x86 architectures which perform FPU computations on
using SSE).

Fixes `num::dec2flt::fast_path_correct` (on x87).
  • Loading branch information
ranma42 committed May 13, 2016
1 parent a581c82 commit 78124a2
Show file tree
Hide file tree
Showing 2 changed files with 42 additions and 7 deletions.
2 changes: 2 additions & 0 deletions src/libcore/lib.rs
Expand Up @@ -61,7 +61,9 @@
#![cfg_attr(not(stage0), deny(warnings))]

#![feature(allow_internal_unstable)]
#![feature(asm)]
#![feature(associated_type_defaults)]
#![feature(cfg_target_feature)]
#![feature(concat_idents)]
#![feature(const_fn)]
#![feature(cfg_target_has_atomic)]
Expand Down
47 changes: 40 additions & 7 deletions src/libcore/num/dec2flt/algorithm.rs
Expand Up @@ -32,19 +32,49 @@ fn power_of_ten(e: i16) -> Fp {
Fp { f: sig, e: exp }
}

#[cfg(any(not(target_arch="x86"), target_feature="sse2"))]
mod fpu_precision {
pub fn set_precision<T>() { }
}

#[cfg(all(target_arch="x86", not(target_feature="sse2")))]
mod fpu_precision {
use mem::size_of;
use ops::Drop;

pub struct FPUControlWord(u16);

fn set_cw(cw: u16) {
unsafe { asm!("fldcw $0" :: "m" (cw)) :: "volatile" }
}

pub fn set_precision<T>() -> FPUControlWord {
let cw = 0u16;
let cw_precision = match size_of::<T>() {
4 => 0x0000, // 32 bits
8 => 0x0200, // 64 bits
_ => 0x0300, // default, 80 bits
};
unsafe { asm!("fnstcw $0" : "=*m" (&cw)) ::: "volatile" }
set_cw((cw & 0xFCFF) | cw_precision);
FPUControlWord(cw)
}

impl Drop for FPUControlWord {
fn drop(&mut self) {
set_cw(self.0)
}
}
}

/// The fast path of Bellerophon using machine-sized integers and floats.
///
/// This is extracted into a separate function so that it can be attempted before constructing
/// a bignum.
///
/// The fast path crucially depends on arithmetic being correctly rounded, so on x86
/// without SSE or SSE2 it will be **wrong** (as in, off by one ULP occasionally), because the x87
/// FPU stack will round to 80 bit first before rounding to 64/32 bit. However, as such hardware
/// is extremely rare nowadays and in fact all in-tree target triples assume an SSE2-capable
/// microarchitecture, there is little incentive to deal with that. There's a test that will fail
/// when SSE or SSE2 is disabled, so people building their own non-SSE copy will get a heads up.
///
/// FIXME: It would nevertheless be nice if we had a good way to detect and deal with x87.
/// without SSE or SSE2 it requires the precision of the x87 FPU stack to be changed
/// so that it directly rounds to 64/32 bit.
pub fn fast_path<T: RawFloat>(integral: &[u8], fractional: &[u8], e: i64) -> Option<T> {
let num_digits = integral.len() + fractional.len();
// log_10(f64::max_sig) ~ 15.95. We compare the exact value to max_sig near the end,
Expand All @@ -60,6 +90,9 @@ pub fn fast_path<T: RawFloat>(integral: &[u8], fractional: &[u8], e: i64) -> Opt
if f > T::max_sig() {
return None;
}

let _cw = fpu_precision::set_precision::<T>();

// The case e < 0 cannot be folded into the other branch. Negative powers result in
// a repeating fractional part in binary, which are rounded, which causes real
// (and occasioally quite significant!) errors in the final result.
Expand Down

0 comments on commit 78124a2

Please sign in to comment.