viable/strict/1789545173: [cuda][nccl] Drop the dead `NCCL_ALLTOALL_SUPPORTED` arm (#196400)
PyTorch 合并 PR #196400，在 nccl.cpp 的 all2all_single_equal_split 中删除已失效的 defined(NCCL_ALLTOALL_SUPPORTED) 分支。证据称该宏在仓库中已无定义：D115674668 从 nccl.h.in 的 v2_29 与 v2_30 中删除了 #define，使旧版 PyTorch 不再选中大写 T 的 ncclAllToAll API，因此该条件恒为 0，#elif 退化为相邻的 NCCL_VERSION_CODE 检查。此前 D115678532 清理了 AllToAllTest.cc 的 #ifdef，D115678533 将 nccl-tests-suite 基准改按 NCCL_VERSION_CODE 判定，nccl.cpp 是这轮清理的第三个文件。
发展脉络
- 首次出现viable/strict/1789545173: [cuda][nccl] Drop the dead `NCCL_ALLTOALL_SUPPORTED` arm (#196400)PyTorch Core
- 行业反馈trunk/7083f3a16277e0e4f58b5064dcc5d9840bf2c24c: [cuda][nccl] Drop the dead `NCCL_ALLTOALL_SUPPORTED` arm (#196400)PyTorch Core
- 当前判断判断：这属于大型开源训练框架与通信库之间的接口清理，反映 NCCL/NCCLX 版本演进中弃用 API 的迁移成本被分摊到下游框架。它不改变模型能力或成本结构，但影响依赖 PyTorch 源码构建的发行版与自编译用户的兼容性预期。可验证下一信号：后续 PyTorch 版本说明或 NCCL 侧文档是否正式标注大写 T ncclAllToAll 为弃用。Agent Pulse · 分析
PyTorch 核心仓库合并 #196400，移除 nccl.cpp 中 all2all_single_equal_split 里已死的 NCCL_ALLTOALL_SUPPORTED 分支。证据显示该宏在仓库内已无定义：D115674668 从 nccl.h.in 的 v2_29 和 v2_30 删除 #define，目的是让针对新版 NCCLX 编译的旧 PyTorch 不再选择大写 T 的 ncclAllToAll API，于是该条件恒为 0，#elif 只剩旁边的 NCCL_VERSION_CODE 判断。该行是 all-to-all 去重栈的遗留：D115510409 曾有意保留该宏，理由是 NCCLX 仍会定义它、且它仍能表示“该 NCCL 具备 all-to-all 集合通信”，但次日 D115674668 使这一前提失效。后续 D115678532 清理 AllToAllTest.cc 的死 #ifdef，D115678533 将 nccl-tests-suite 基准改按 NCCL_VERSION_CODE 判定，nccl.cpp 此前未被回访。证据还指出，保留该条件不只是整洁问题：只要它存在，任何地方重新引入该 #define 都会静默复活已被弃用的大写 T 路径。
这是编译期分支的收敛：宏被删除后，条件恒为 0，实际生效路径由 NCCL_VERSION_CODE 决定，行为等价但消除了隐式开关。判断：真正的风险不在当前构建，而在于残留条件会让未来重新定义宏时静默切回已弃用 API。可验证下一信号：仓库内是否仍存在 NCCL_ALLTOALL_SUPPORTED 的引用，以及 nccl.h.in 是否再次出现该 #define。
判断：这属于大型开源训练框架与通信库之间的接口清理，反映 NCCL/NCCLX 版本演进中弃用 API 的迁移成本被分摊到下游框架。它不改变模型能力或成本结构，但影响依赖 PyTorch 源码构建的发行版与自编译用户的兼容性预期。可验证下一信号：后续 PyTorch 版本说明或 NCCL 侧文档是否正式标注大写 T ncclAllToAll 为弃用。
判断：对自建训练栈的团队，价值在于减少构建期隐式分支带来的行为不确定性，降低升级 NCCL 时的排查成本。可验证下一信号：升级到含该改动的版本后，all-to-all 相关测试与基准是否仍按预期通过。
判断：若该宏不再被重新引入，此类条件分支会继续从 PyTorch 代码中消失，版本判断将统一收敛到 NCCL_VERSION_CODE。可验证下一信号：后续 PR 是否继续清理同类遗留宏，或出现回退该删除的讨论。