viable/strict/1787313317: Remove the caffe2/core/macros.h placeholder (#194230)
PyTorch 的 PR #194230 移除了 caffe2/core/macros.h 占位文件。该文件原本只有两行，说明真实内容由 cmake 生成。cmake/Codegen.cmake:47 将生成的文件写入 ${CMAKE_BINARY_DIR}，build.bzl:16 写入 Bazel 输出树，只有生成的文件会被安装。没有代码包含占位文件。caffe2/core/common.h 以完整路径 "caffe2/core/macros.h" 引用，导致引号包含搜索自身目录时查找 caffe2/core/caffe2/core/macros.h 而失败，随后 -I 搜索找到构建树（位于源码树之前）。该 PR 由 AI 助手辅助编写，已合并。
Development
- First Reportviable/strict/1787313317: Remove the caffe2/core/macros.h placeholder (#194230)PyTorch Core
- Industry Responsetrunk/563e3c7033dcbee4ebd30da915b75dc5ba0f5ff1: Revert "Remove the caffe2/core/macros.h placeholder (#194230)"PyTorch Core
- Industry Responseviable/strict/1787505913: Remove the caffe2/core/macros.h placeholder (#194230)PyTorch Core
- Current AssessmentPyTorch 作为主流深度学习框架，其代码清理工作反映了开源项目在维护构建系统时的常见挑战。AI 辅助编写 PR 的实践表明，AI 工具正在参与基础设施级别的代码维护，这可能提高效率，但也需要人工审查以确保正确性。Agent Pulse · analysis
PyTorch 仓库移除了 caffe2/core/macros.h 占位文件，该文件仅包含两行注释，说明真实内容由 cmake 生成。生成的文件被写入构建目录（cmake/Codegen.cmake:47 写入 ${CMAKE_BINARY_DIR}，build.bzl:16 写入 Bazel 输出树），且只有生成的文件会被安装。没有任何代码包含占位文件。caffe2/core/common.h 以完整路径 "caffe2/core/macros.h" 引用，导致引号包含搜索自身目录时查找 caffe2/core/caffe2/core/macros.h 而失败，随后 -I 搜索找到构建树（位于源码树之前）。该 PR 由 AI 助手辅助编写，已合并。
该变更揭示了构建系统中的一个微妙问题：当源文件以完整路径引用头文件时，引号包含搜索会先查找自身目录，导致找不到占位文件，但 -I 搜索会找到构建树中的生成文件。移除占位文件后，构建系统更干净，但依赖生成文件的正确顺序。这提示开发者注意头文件搜索路径的优先级，以及占位文件可能掩盖的构建问题。
PyTorch 作为主流深度学习框架，其代码清理工作反映了开源项目在维护构建系统时的常见挑战。AI 辅助编写 PR 的实践表明，AI 工具正在参与基础设施级别的代码维护，这可能提高效率，但也需要人工审查以确保正确性。
该变更降低了构建系统的复杂性，减少了占位文件可能导致的混淆，从而降低维护成本。对于依赖 PyTorch 的企业，这有助于减少构建错误，提高开发效率。
未来，类似 AI 辅助的代码清理 PR 可能会更频繁，但需要更严格的测试和审查流程。构建系统的简化可能减少维护负担，但需警惕生成文件与源码树之间的路径冲突。