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 fails on abc/src/misc/extra/extraUtilUtil.c: "ISO C++17 does not allow 'register' storage class specifier" #3881

Closed
salfter opened this issue Aug 7, 2023 · 3 comments

Comments

@salfter
Copy link

salfter commented Aug 7, 2023

Version

0.32

On which OS did this happen?

Linux

Reproduction Steps

I maintain a Gentoo Linux ebuild for Yosys and was updating to the latest version. Compilation fails during the ABC build:

[ 94%] ABC: `` Compiling: /src/misc/mvc/mvcCover.c
src/misc/extra/extraUtilUtil.c:100:5: error: ISO C++17
      does not allow 'register' storage class specifier [-Wregister]
    register int c;
    ^~~~~~~~~
src/misc/extra/extraUtilUtil.c:101:5: error: ISO C++17
      does not allow 'register' storage class specifier [-Wregister]
    register const char *place;
    ^~~~~~~~~
2 errors generated.
make[1]: *** [Makefile:177: src/misc/extra/extraUtilUtil.o] Error 1

This error also prevents older versions of Yosys from building. I brought up an issue with the ABC project and got it fixed in berkeley-abc/abc@927b60b. I know that Yosys pulls ABC from its own fork, but you might want to consider rebasing against upstream.

Expected Behavior

Yosys builds properly with current versions of the toolchain.

Actual Behavior

It doesn't.

@salfter salfter added the pending-verification This issue is pending verification and/or reproduction label Aug 7, 2023
@whitequark
Copy link
Member

Related/caused by #2011.

@whitequark whitequark added bug ABC and removed pending-verification This issue is pending verification and/or reproduction labels Aug 7, 2023
@whitequark
Copy link
Member

Should be fixed by #4234.

@salfter
Copy link
Author

salfter commented Feb 26, 2024

Got rid of the patch I had been using...looks like it builds fine without it now. Thanks!

gentoo-repo-qa-bot pushed a commit to gentoo-mirror/salfter that referenced this issue Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants