Skip to content

std.math: Implement floor for ibmExtended reals.#5822

Merged
dlang-bot merged 1 commit intodlang:masterfrom
ibuclaw:ppcfloor
Apr 6, 2018
Merged

std.math: Implement floor for ibmExtended reals.#5822
dlang-bot merged 1 commit intodlang:masterfrom
ibuclaw:ppcfloor

Conversation

@ibuclaw
Copy link
Member

@ibuclaw ibuclaw commented Oct 29, 2017

This is just a slight extension of #5820. Tested against PPC64/LE.

https://github.com/dlang/phobos/pull/5822/files?w=1

@dlang-bot
Copy link
Contributor

dlang-bot commented Oct 29, 2017

Thanks for your pull request, @ibuclaw!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub fetch digger
dub run digger -- build "master + phobos#5822"

@quickfur
Copy link
Member

quickfur commented Feb 5, 2018

ping reviewers: this PR has been rotting here for far too long. We need someone familiar with numerics to take a look!

@quickfur
Copy link
Member

quickfur commented Feb 5, 2018

ping @klickverbot

@ibuclaw
Copy link
Member Author

ibuclaw commented Mar 17, 2018

@redstar - Oh, and this you may be interested in also.

@PetarKirov
Copy link
Member

Needs a rebase after #5823 is merged.

@JackStouffer
Copy link
Contributor

Merging based on qualified review

@wilzbach wilzbach added the Merge:72h no objection -> merge The PR will be merged if there are no objections raised. label Mar 30, 2018
@n8sh
Copy link
Member

n8sh commented Mar 30, 2018

@wilzbach The reason this wasn't merged three days ago isn't due to anyone's objection, but because PR #5820 also by @ibuclaw was auto-merged first and it conflicts with this PR. As soon as the conflicts are resolved this PR can be merged.

@wilzbach
Copy link
Contributor

Oh I was just marking the PR, s.t. I didn't forget to rebase it later (done that now).
BTW this helps when re-reviewing:

https://github.com/dlang/phobos/pull/5822/files?w=1

@ibuclaw
Copy link
Member Author

ibuclaw commented Apr 1, 2018

@wilzbach didn't quite rebase it correctly, now I'm back I've fixed that. ;-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

math Merge:auto-merge Merge:72h no objection -> merge The PR will be merged if there are no objections raised.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants