Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rustc_apfloat: Double::mul_add_r panic with specific values #93225

Closed
wants to merge 1 commit into from

Conversation

mjdr
Copy link

@mjdr mjdr commented Jan 23, 2022

Code related to issue #93224

@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Jan 23, 2022
@rust-highfive
Copy link
Collaborator

Changes rustc_apfloat. rustc_apfloat is currently in limbo and you almost certainly don't want to change it (see #55993).

cc @eddyb

@rust-highfive
Copy link
Collaborator

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @wesleywiser (or someone else) soon.

Please see the contribution instructions for more information.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 23, 2022
@rust-log-analyzer
Copy link
Collaborator

The job x86_64-gnu-llvm-12 failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)
running 50 tests
..............................F...................
failures:

---- issue_93224 stdout ----
thread 'issue_93224' panicked at 'assertion failed: `(left == right)`
 right: `0`', compiler/rustc_apfloat/src/ieee.rs:2545:17
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace



failures:
    issue_93224

test result: FAILED. 49 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

error: test failed, to rerun pass '-p rustc_apfloat --test ieee'


command did not execute successfully: "/checkout/obj/build/x86_64-unknown-linux-gnu/stage0/bin/cargo" "test" "--target" "x86_64-unknown-linux-gnu" "-Zbinary-dep-depinfo" "-j" "16" "--release" "--locked" "--color" "always" "--features" " llvm" "--manifest-path" "/checkout/compiler/rustc/Cargo.toml" "-p" "rustc_apfloat" "--" "--quiet"


Build completed unsuccessfully in 0:23:44

@JohnCSimon JohnCSimon added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Feb 27, 2022
@wesleywiser wesleywiser added the I-compiler-nominated Nominated for discussion during a compiler team meeting. label Mar 4, 2022
@wesleywiser
Copy link
Member

Nominating for discussion at the compiler team meeting because I'm not sure what to do with apfloat changes in light of #55993.

@apiraino apiraino removed the I-compiler-nominated Nominated for discussion during a compiler team meeting. label Mar 16, 2022
@wesleywiser wesleywiser added the S-blocked Status: Blocked on something else such as an RFC or other implementation work. label Mar 28, 2022
@JohnCSimon JohnCSimon added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 24, 2022
@JohnCSimon JohnCSimon added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels May 22, 2022
@Dylan-DPC Dylan-DPC removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 24, 2022
@apiraino
Copy link
Contributor

This is waiting on #96784 (if i am not mistaken)

@eddyb
Copy link
Member

eddyb commented Jul 14, 2023

I posted a longer update to #55993 (comment) regarding the progress with the whole rustc_apfloat situation, but of note for this PR is that the WIP repo already has this bug fixed.

@wesleywiser
Copy link
Member

Hey @mjdr, thanks for your PR! As @eddyb noted, this has been fixed by the new version of rustc_apfloat that #113843 switched us to using.

Thanks again for your efforts! 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-blocked Status: Blocked on something else such as an RFC or other implementation work. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants