-
-
Notifications
You must be signed in to change notification settings - Fork 10.9k
ENH, BLD: Support for the NVIDIA HPC SDK nvfortran compiler #17344
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
Conversation
numpy/distutils/fcompiler/pg.py
Outdated
@@ -12,7 +12,7 @@ class PGroupFCompiler(FCompiler): | |||
|
|||
compiler_type = 'pg' | |||
description = 'Portland Group Fortran Compiler' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we update this description too?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
compiler_type
might be also changed to "nv", but it will break compatibility for build script that uses --fcompiler=pg
.
Should we create a all new compiler type nv.py
and leave pg.py
as is for older versions ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that's reasonable - but I think you need to also check distutils/fcompiler/__init__.py
and add an entry to tests/test_public_api.py
for the new compiler. There may be other places worth checking.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the hints.
I removed the modifications on the pg.py
files and created the nv.py
one instead.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok to me, but I don't know if there are specific flags for other systems - I see that was the case for pgfortran
.
I guess the flags would be the same as those for pgfortran on other platforms. |
Co-authored-by: Eric Wieser <wieser.eric@gmail.com>
Thanks @danbeibei . |
Add support for the nvfortran compiler, a version of pgfortran.
Closes #17341.