Skip to content

Conversation

@lgrvs
Copy link

@lgrvs lgrvs commented Apr 13, 2018

No description provided.

lgrvs added 2 commits April 13, 2018 22:04
*Working* cpp implementation.
Requires gmp lib (<gmpxx.h>)
@CLAassistant
Copy link

CLAassistant commented Apr 13, 2018

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@arguiot arguiot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@arguiot arguiot requested a review from lukalafaye April 13, 2018 12:09
Copy link
Member

@arguiot arguiot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Having problem running this on other platforms. Requesting a patch

@arguiot
Copy link
Member

arguiot commented Apr 14, 2018

@el-famos0 Could you add a Makefile in your PR?

Make sure you include the following command:

clang++ -L /opt/local/lib -I /opt/local/include -lgmpxx -lgmp main.cpp

Added basic arguments support.
Multi-threading not implemented yet...
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

Successfully merging this pull request may close these issues.

3 participants