Skip to content

Conversation

JorgeG94
Copy link

@JorgeG94 JorgeG94 commented Oct 8, 2025

This fixes compiling test-drive with nvfortran 25.9. This seems to be like a compiler bug, I will create an MRE and report

@JorgeG94
Copy link
Author

JorgeG94 commented Oct 8, 2025

Copy link
Member

@awvwgk awvwgk left a comment

Choose a reason for hiding this comment

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

Thanks for sharing. Surprising that the interface for the procedure pointer inside the type needs to be visible in the abstract interface, it seems to break the encapsulation the derived type should provide. Fine with including this if it makes test-drive available with nvfortran.

@JorgeG94
Copy link
Author

JorgeG94 commented Oct 9, 2025

Thanks for sharing. Surprising that the interface for the procedure pointer inside the type needs to be visible in the abstract interface, it seems to break the encapsulation the derived type should provide. Fine with including this if it makes test-drive available with nvfortran.

It also took me by surprise. I've logged a ticket with nvidia since it might be a bug on their side and we'll need to revert this later. I can leave a comment in the code for this

@jvdp1
Copy link
Member

jvdp1 commented Oct 9, 2025

Could it be encapsulated in a CPP directive:

  import ...
#ifdef __NVCOMPILER
  import ...
#endif
 ....

@JorgeG94
Copy link
Author

JorgeG94 commented Oct 9, 2025

Could it be encapsulated in a CPP directive:

  import ...
#ifdef __NVCOMPILER
  import ...
#endif
 ....

See: 81e0bb4

@JorgeG94
Copy link
Author

JorgeG94 commented Oct 9, 2025

Thanks for the report and nice reproducer. I was able to recreate the error here and have filed a problem report, TPR #37894. I’ve asked engineer to investigate especially since this is a regression.

-Mat

Bug report has been filed with nvidia :) they've picked it up.

@JorgeG94
Copy link
Author

The CI appears to be dead

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

Successfully merging this pull request may close these issues.

3 participants