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

Compile failure on recent master #277

Closed
dbogatov opened this issue Nov 15, 2023 · 4 comments
Closed

Compile failure on recent master #277

dbogatov opened this issue Nov 15, 2023 · 4 comments

Comments

@dbogatov
Copy link

[ 34%] Building C object src/CMakeFiles/relic.dir/pp/relic_pp_norm.c.o
[ 34%] Building C object src/CMakeFiles/relic.dir/pc/relic_pc_core.c.o
[ 34%] Building C object src/CMakeFiles/relic.dir/pc/relic_pc_exp.c.o
/Users/bogatov/Desktop/bu/ipfre/relic/src/pc/relic_pc_exp.c:152:2: error: call to undeclared function 'fp2_exp_cyc_gls'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
        RLC_CAT(RLC_GT_LOWER, exp_cyc_gls)(c, a, b);
        ^
/Users/bogatov/Desktop/bu/ipfre/relic/include/relic_util.h:158:25: note: expanded from macro 'RLC_CAT'
#define RLC_CAT(A, B)                   _RLC_CAT(A, B)
                                        ^
/Users/bogatov/Desktop/bu/ipfre/relic/include/relic_util.h:159:26: note: expanded from macro '_RLC_CAT'
#define _RLC_CAT(A, B)                  A ## B
                                        ^
<scratch space>:160:1: note: expanded from here
fp2_exp_cyc_gls
^
1 error generated.
make[2]: *** [src/CMakeFiles/relic.dir/pc/relic_pc_exp.c.o] Error 1
make[1]: *** [src/CMakeFiles/relic.dir/all] Error 2
make: *** [all] Error 2

^^ for both arm and x86 on commit 6b90545

Preset gmp-pbc-ss1536.sh but also saw on a few other GMP presets.

@dfaranha
Copy link
Contributor

This one is fixed, it was something silly on my part! Now trying to reproduce in other presets.

@dbogatov
Copy link
Author

Oh thanks for the quick response! Let me know if there is a commit or branch I can work off, or when you post a fix!

@dfaranha
Copy link
Contributor

It's pushed to main already, commit c5ac648

@dbogatov
Copy link
Author

It fixes it, thanks!

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

2 participants