[clang] [llvm] MC: Move BinutilsVersion from TargetOptions to MCTargetOptions (PR #221435)

Fangrui Song via cfe-commits cfe-commits at lists.llvm.org
Sat Sep 5 15:59:02 PDT 2026


MaskRay wrote:

> > BinutilsVersion has no codegen use
> 
> This seems false. It has a bunch of uses in llvm/lib/CodeGen. Real justification:
> 
> TargetOptions::BinutilsVersion only seeds MCAsmInfo, which already takes MCTargetOptions, so drop the indirection.

Looks good once the PR description is changed.

https://github.com/llvm/llvm-project/pull/221435


More information about the cfe-commits mailing list