from_str_r moves exactly representable decimals by one ULP under TowardZero/TowardNegative (port of llvm#228928)
还没有人认领这个 Issue。
评估
调研方向
Start by reading the from_str_r implementation and the existing floating point conversion tests; the issue gives reproducible decimal inputs and expected bit patterns for Single and Double under directed rounding modes. Add regression cases covering the reported exact and near-boundary conversions, then verify the resulting values and status flags across the affected formats and rounding modes. The issue says the fix likely depends on an upstream LLVM change, so check that defect’s status before estimating implementation work.
由索引模型根据 Issue 内容生成。
描述
This was identified by comparing floating point number conversion results across a few open-source implementations. The implementation of from_str_r incorrectly rounds some exactly representable numbers, however only in TowardZero/TowardNegative rounding modes which do not appear to be used yet. That makes this lower priority but opening for awareness - feel free to close if this is expected/intended.
Below is the full AI-generated summary:
from_str_r treats some exactly representable decimal strings as inexact. Under TowardZero and TowardNegative (and TowardPositive for negative inputs) it then returns the value one ULP below the exact one. The status is INEXACT in every mode, including NearestTiesToEven, where the value itself is right.
This is the LLVM APFloat defect reported as https://github.com/llvm/llvm-project/issues/228928, carried over by the port. rustc_apfloat 0.2.3 (+llvm-462a31f5a5ab) fails on different inputs than current LLVM does (the single-precision example in the LLVM issue converts correctly here), but the pattern is the same.
[dependencies]
rustc_apfloat = "=0.2.3"
use rustc_apfloat::ieee::{Double, Single};
use rustc_apfloat::{Float, Round};
fn check<F: Float>(label: &str, text: &str, round: Round, want: u128) -> bool {
let r = F::from_str_r(text, round).unwrap();
let got = r.value.to_bits();
let ok = got == want;
println!("{label:<24} {got:#018x} want {want:#018x} {:?}{}", r.status, if ok { "" } else { " <- WRONG" });
ok
}
fn main() {
// (2^23 + 1) * 2^-45: exactly the binary32 value 0x34800001
let single = "0.000000238418607523271930404007434844970703125";
// 2^11 + 2^-41: exactly the binary64 value 0x40A0000000000001
let double = "2048.00000000000045474735088646411895751953125";
let mut ok = true;
for (name, round) in [
("NearestTiesToEven", Round::NearestTiesToEven),
("TowardZero", Round::TowardZero),
("TowardNegative", Round::TowardNegative),
("TowardPositive", Round::TowardPositive),
] {
ok &= check::<Single>(&format!("Single {name}"), single, round, 0x34800001);
ok &= check::<Double>(&format!("Double {name}"), double, round, 0x40A0000000000001);
}
std::process::exit(if ok { 0 } else { 1 });
}
Output (rustc 1.99.0, Linux aarch64):
Single NearestTiesToEven 0x0000000034800001 want 0x0000000034800001 Status(INEXACT)
Double NearestTiesToEven 0x40a0000000000001 want 0x40a0000000000001 Status(INEXACT)
Single TowardZero 0x0000000034800000 want 0x0000000034800001 Status(INEXACT) <- WRONG
Double TowardZero 0x40a0000000000000 want 0x40a0000000000001 Status(INEXACT) <- WRONG
Single TowardNegative 0x0000000034800000 want 0x0000000034800001 Status(INEXACT) <- WRONG
Double TowardNegative 0x40a0000000000000 want 0x40a0000000000001 Status(INEXACT) <- WRONG
Single TowardPositive 0x0000000034800001 want 0x0000000034800001 Status(INEXACT)
Double TowardPositive 0x40a0000000000001 want 0x40a0000000000001 Status(INEXACT)
Scope: a sweep over decimal strings into Double, Single, Half, BFloat, Float8E5M2, Float8E4M3FN and Quad in all five modes found 1,268 exactly representable inputs moved one step in the rounding direction, in Double, Single, BFloat and Quad. It also found 84 inexact inputs within a tiny fraction of an ULP of a Quad value, rounded as if they were on the other side of it (e.g. 4.5326225836066175776471072707593525867497e-4965 under TowardZero gives 6 × 2⁻¹⁶⁴⁹⁴ instead of 7 × 2⁻¹⁶⁴⁹⁴). No other differences turned up in 151,415 conversions. Half and the FP8 types had none (LLVM 21.1.8 does fail for IEEEhalf).
Since rustc only folds literals with NearestTiesToEven, compiled code is not affected. This matters to direct users of the crate who pass a directed mode, and to anyone relying on the status flag. The fix most likely belongs in LLVM first; this issue is here so it gets ported when that happens.
Use the following links to see the expected conversion results:
- 主要语言
- Rust
- 星标
- 23
- 派生
- 18
- 平均合并
- 2 天 13 小时
- 30 天内合并 PR
- 2
环境准备
这个项目没有提供开发容器、Dockerfile 或贡献指南,环境需要你自己搭建:先看它的 README,通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
rust-lang/rustc_apfloat 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 70/100
rust-lang/rustc_apfloat#44 · 3 条评论 ·
-
Overflow flag when result is larget finite value可能已有人在做 @Garfield1002 于 51 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 84/100
rust-lang/rustc_apfloat#36 ·
-
难度 2/5 1-3 小时 新手友好度 45/100
rust-lang/rustc_apfloat#20 ·
-
难度 4/5 3-5 天 新手友好度 35/100
rust-lang/rustc_apfloat#14 · 1 条评论 ·
-
难度 2/5 1-3 小时 新手友好度 45/100
rust-lang/rustc_apfloat#10 · 7 条评论 ·
查看 rust-lang/rustc_apfloat 的全部 Issue
相似的 Issue
-
documentation priority:p2 type:chore
难度 2/5 1-3 小时 新手友好度 73/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 68/100
facebookincubator/scrut#60 ·
-
dispatch:sol-high documentation
难度 1/5 1-3 小时 新手友好度 85/100
carrick-sh/carrick#768 · 1 条评论 ·
维护者通常 1 天内回复
-
correctness postgres tursopg
难度 2/5 1-3 小时 新手友好度 76/100
tursodatabase/turso#10442 ·
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 82/100
ankitects/anki#5864 · 1 个 reaction ·
维护者通常 2 天内回复