Skip to content

Conversation

@Binyang2014
Copy link
Contributor

No description provided.

@Binyang2014 Binyang2014 marked this pull request as ready for review September 10, 2025 16:34
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adapts the NCCL implementation to support PyTorch 2.6 by adding the new ncclAllToAllv function. PyTorch 2.6 requires this collective communication primitive for more flexible all-to-all operations with variable send/receive counts.

  • Added ncclAllToAllv function declaration and implementation
  • Provided comprehensive documentation for the new API
  • Implemented stub functionality that returns unavailable status

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
apps/nccl/include/mscclpp/nccl.h Added function declarations and documentation for ncclAllToAllv and pncclAllToAllv
apps/nccl/src/nccl.cu Added stub implementation of ncclAllToAllv that returns unavailable status

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Binyang2014 Binyang2014 enabled auto-merge (squash) September 10, 2025 20:36
@Binyang2014 Binyang2014 merged commit d946c45 into main Sep 10, 2025
14 checks passed
@Binyang2014 Binyang2014 deleted the binyli/rccl branch September 10, 2025 20:40
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

SYSTEM_READY >> ...MS