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

Division Node is missing #73

Open
christoph-hart opened this issue Feb 23, 2017 · 6 comments
Open

Division Node is missing #73

christoph-hart opened this issue Feb 23, 2017 · 6 comments

Comments

@christoph-hart
Copy link

I realized there is no node for division. Is there a reason for this (I am wondering because it's such a basic operation).

@MikeHopcroft
Copy link
Contributor

Let me investigate the reason for this. If I recall correctly, the integer divide instruction had some requirements for input/output registers that were harder to satisfy. I believe the node can be added, but that we just did all of the binary operators that didn't have the register restriction. I will see how much work it would be to implement this.

As a workaround, you can call a C-function that either computes 1/x or x/y.

@ConsoleTVs
Copy link

Still wondering the same :)

@mackrol
Copy link

mackrol commented Sep 6, 2018

Any progress in division?

@MikeHopcroft
Copy link
Contributor

A the moment, this work is prioritized lower than work we're doing in the BitFunnel repo. Could you give us an idea of how important this work is to your use case, as this might boost the priority for divide?

@mackrol
Copy link

mackrol commented Sep 7, 2018

I use div function for now but that's suboptimal in terms of register allocation. I'm just curious / worried whether the project is alive.

@josephjaspers
Copy link

I am also requesting support for a division node. Thanks for your work so far though!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants