Skip to content

fix: add missing #include <memory> for std::unique_ptr#1

Open
LorrinWWW wants to merge 1 commit into
meituan-longcat:mainfrom
LorrinWWW:fix/add-memory-include
Open

fix: add missing #include <memory> for std::unique_ptr#1
LorrinWWW wants to merge 1 commit into
meituan-longcat:mainfrom
LorrinWWW:fix/add-memory-include

Conversation

@LorrinWWW

@LorrinWWW LorrinWWW commented Feb 11, 2026

Copy link
Copy Markdown

Problem

The hopper_grouped_gemm.h file uses std::unique_ptr but was missing the <memory> header include. This caused compilation errors when building the eps package:

error: qualified name is not allowed
    std::unique_ptr<Impl> impl_;
    ^

error: this declaration has no storage class or type specifier
    std::unique_ptr<Impl> impl_;
    ^

Solution

Added #include <memory> to the header file to properly declare std::unique_ptr.

Testing

  • Built the eps package successfully after this fix
  • The change is minimal and follows C++ standard library best practices

The hopper_grouped_gemm.h file uses std::unique_ptr but was missing
the <memory> header include. This caused compilation errors when
building with CUDA:

  error: qualified name is not allowed
  error: this declaration has no storage class or type specifier

This commit adds the missing include to resolve the compilation issue.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
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.

1 participant