Skip to content

Include iostream in launch.cu#2050

Merged
ericniebler merged 2 commits intoNVIDIA:mainfrom
dkolsen-pgi:bug/missing-iostream
Apr 27, 2026
Merged

Include iostream in launch.cu#2050
ericniebler merged 2 commits intoNVIDIA:mainfrom
dkolsen-pgi:bug/missing-iostream

Conversation

@dkolsen-pgi
Copy link
Copy Markdown
Contributor

examples/nvexec/launch.cu uses std::cout but didn't include <iostream>. In the past this was compiling because one of the Thrust headers (I think) was including <iostream>. But Thrust recently did some more internal header cleanup and removed that indirect include, causing launch.cu to no longer compile.

Add the necessary include of <iostream> so launch.cu compiles again.

`examples/nvexec/launch.cu` uses `std::cout` but didn't include
`<iostream>`.  In the past this was compiling because one of the Thrust
headers (I think) was including `<iostream>`.  But Thrust recently did
some more internal header cleanup and removed that indirect include,
causing `launch.cu` to no longer compile.

Add the necessary include of `<iostream>` so `launch.cu` compiles again.
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Apr 26, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@ericniebler
Copy link
Copy Markdown
Collaborator

/ok to test e6fac08

@ericniebler ericniebler merged commit 1dc8702 into NVIDIA:main Apr 27, 2026
32 checks passed
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.

2 participants