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.

Sign up for GitHub

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

Jump to bottom

solvers: Fix incorrect f_OPHOL declaration #23

Open

Tatsh wants to merge 1 commit into ampl:master from Tatsh:fix-lto

Tatsh commented Dec 2, 2024

gentoo/gentoo#39303
coin-or-tools/ThirdParty-ASL#8

This change fixes the following issue with LTO:

solvers/fgh_read.c:101:15: error: function 'f2_HOL_ASL' redeclared as variable
  101 |  extern char* f_OPHOL;
      |               ^
solvers/rops2.c:1019:1: note: previously declared here
 1019 | f_OPHOL(expr *e A_ASL)
      | ^
lto1: fatal error: errors during merging of translation units
compilation terminated.

glebbelov added a commit that referenced this pull request


          Remove unneeded solvers2/fgh_read.c #23

97e1074

Tatsh force-pushed the fix-lto branch from b0c644b to 7ebb42d Compare

December 3, 2024 07:20


          solvers: Fix incorrect f_OPHOL declaration

e69d96d

Tatsh force-pushed the fix-lto branch from 7ebb42d to e69d96d Compare

December 3, 2024 07:20

Tatsh changed the title ~~solvers: Fix incorrect f_OPHOL declarations~~ solvers: Fix incorrect f_OPHOL declaration

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

Labels

None yet