执行摘要

• 一句话：显式启用 x86_64 的 AVX512/AMX 指令集
• 推荐动作：该 PR 值得阅读，它展示了在构建系统中如何处理指令集兼容性。对于维护者，需要确认发布二进制是否包含这些指令集，以及对旧硬件的策略；对于使用 CPU 后端的用户，建议验证目标 CPU 的指令集支持。

功能与动机

It was recently found that arch=native is set in CPU CMakeLists.txt. So if a binary was built on a host server that does not support AVX512/AMX, the output lacks the instruction set support.

实现拆解

1. 在 sgl-kernel/csrc/cpu/CMakeLists.txt 中，添加架构判断：若 MY_ARCH_DIR 为 x86_64，则使用精细的 AVX512/AMX 标志列表替换 -march=native；否则保持原样。
2. 根据 reviewer 建议修正 CMake 变量引用方式（从 ${MY_ARCH_DIR} 改为 MY_ARCH_DIR）。
3. 无测试配套，但构建体系保证链接，非 x86_64 架构逻辑不变。

关键文件：

• sgl-kernel/csrc/cpu/CMakeLists.txt（模块 构建配置；类别 config；类型 configuration）: 唯一修改文件，通过条件编译为 x86_64 显式启用 AVX512/AMX 指令集，避免依赖构建服务器硬件支持。

关键符号：未识别

评论区精华

mingfeima 指出 CMake 变量引用方式问题，建议使用 if(MY_ARCH_DIR STREQUAL "x86_64") 而非 if(${MY_ARCH_DIR} STREQUAL "x86_64")，以避免变量未定义时的错误。作者采纳并修改。

• CMake 变量引用方式 (correctness): 作者采纳了建议，在后续提交中修改。

风险与影响

• 风险：主要风险在于兼容性：x86_64 架构下强制要求 AVX512/AMX 指令集（通过 -march=x86-64-v4 等标志），若目标 CPU 不支持这些指令集，则二进制无法运行。而之前 -march=native 仅在构建机器支持时才包含这些指令。建议在发布二进制时明确标注所需 CPU 最低配置（如 Intel Sapphire Rapids 或更新）。非 x86_64 架构不受影响。
• 影响：影响范围仅限于 CPU 后端的 x86_64 构建。对于支持 AVX512/AMX 的 Intel CPU（如 Sapphire Rapids 及以上），性能将显著提升。对于不支持的 CPU，用户需回退版本或自行修改编译选项。团队应在 CI 或发布流程中确保构建机器支持这些指令集，以避免生成不可移植的二进制。
• 风险标记：目标 CPU 必须支持 AVX512/AMX, 向后兼容性（旧 CPU 无法运行）

关联脉络

• 暂无明显关联 PR