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

Fix compilation issue with latest PGI compiler #832

Merged
merged 3 commits into from
Nov 27, 2020
Merged

Fix compilation issue with latest PGI compiler #832

merged 3 commits into from
Nov 27, 2020

Conversation

iomaganaris
Copy link
Member

Edit the compilerfeatures.h so that the proper header file is included when we compiling NEURON with latest PGI compiler (20.9)

  • Same order as CoreNEURON (corresponding file is in external/Random123/include/Random123/features/compilerfeatures.h)
  • The error I got was error: identifier "__uint128_t" is undefined in src/Random123/philox.h because __uint128_t is not defined in PGI

when we compiling neuron with PGI 20.9 compiler

- Same order as CoreNEURON (corresponding file is in external/Random123/include/Random123/features/compilerfeatures.h)
- The error I got was `error: identifier "__uint128_t" is undefined` in src/Random123/philox.h because `__uint128_t` is not defined in PGI
@iomaganaris iomaganaris added bug build system building Issue related to build/compilation labels Nov 19, 2020
Copy link
Member

@pramodk pramodk left a comment

Choose a reason for hiding this comment

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

LGTM

@pramodk
Copy link
Member

pramodk commented Nov 26, 2020

@alexsavulescu : any idea why this PR should fail windows CI?

@alexsavulescu
Copy link
Member

alexsavulescu commented Nov 27, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug build system building Issue related to build/compilation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants