Skip to content

Conversation

@schweitzpgi
Copy link
Collaborator

Fixes kernel_builder to use the correct declaration of altLaunchKernel and pass it the correct arguments.

Description

Fixes kernel_builder to use the correct declaration of altLaunchKernel
and pass it the correct arguments.

Signed-off-by: Eric Schweitz <[email protected]>
@schweitzpgi schweitzpgi enabled auto-merge (squash) October 6, 2025 17:28
Signed-off-by: Eric Schweitz <[email protected]>
#include "common/ThunkInterface.h"
#include "cudaq/remote_capabilities.h"
#include "cudaq/utils/cudaq_utils.h"
#include "nvqpp_interface.h"
Copy link
Collaborator

@khalatepradnya khalatepradnya Oct 6, 2025

Choose a reason for hiding this comment

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

With the addition of this new header file, those declarations in this file can be removed? (lines 239-258)

github-actions bot pushed a commit that referenced this pull request Oct 6, 2025
@github-actions
Copy link

github-actions bot commented Oct 6, 2025

CUDA Quantum Docs Bot: A preview of the documentation can be found here.

Copy link
Collaborator

@khalatepradnya khalatepradnya left a comment

Choose a reason for hiding this comment

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

LGTM.
One comment on potential to further clean-up duplicate declarations.

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