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

Tutorial, linear regression implementation #8

Open
wants to merge 72 commits into
base: obliv-c
Choose a base branch
from

Conversation

havron
Copy link

@havron havron commented Aug 6, 2015

obliv-c/tutorial/olinReg has not been able to compile when using the newest CIL version; it has only successfully compiled and ran using the older CIL version that my fork is using.

@samee
Copy link
Owner

samee commented Aug 8, 2015

Unfortunately, I cannot merge code into the new version if it does not compile with the new version. The errors right now are caused by two issues:

  • Obliv-C no longer supports the use of yaoGateCount() after the protocol has finished. So just disabling that one line fixes the "Undefined function" error.
  • CIL seems to be generating "goto" commands whenever you use logical &&, which is causing problems. This is a bug on my side, which I will fix. But for now, you can just change the Makefile to compile linReg.c separately with gcc -c linReg.c and just link it in.

This will fix the issue for now.

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.

2 participants