[all-commits] [llvm/llvm-project] fa7e69: [gn] Make RISCV/MCTargetDesc depend on RISCV/Targe...

[Nico Weber via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: fa7e69561c86f898537b80787250ba6673d810ff
      https://github.com/llvm/llvm-project/commit/fa7e69561c86f898537b80787250ba6673d810ff
  Author: Nico Weber <thakis at chromium.org>
  Date:   2025-04-30 (Wed, 30 Apr 2025)

  Changed paths:
    M llvm/utils/gn/secondary/llvm/lib/Target/RISCV/MCTargetDesc/BUILD.gn

  Log Message:
  -----------
  [gn] Make RISCV/MCTargetDesc depend on RISCV/TargetInfo

RISCVMCTargetDesc.cpp calls getTheRISCV{32,64}Target(), which is in TargetInfo.
Looks like this dependency has been missing for a long time.


  Commit: e66eb836e67f2fb0e92b62029f779dad49f5b22b
      https://github.com/llvm/llvm-project/commit/e66eb836e67f2fb0e92b62029f779dad49f5b22b
  Author: Nico Weber <thakis at chromium.org>
  Date:   2025-04-30 (Wed, 30 Apr 2025)

  Changed paths:
    M llvm/utils/gn/secondary/clang/unittests/Frontend/BUILD.gn

  Log Message:
  -----------
  [gn] Make FrontendTests depend on AllTargetsDescs

The CMake build merged most clang unittests into a single binary.
The gn build doesn't (yet?) do this, but as part of that change,
FrontendTests somehow picked up a dependency on AllTargetsDescs,
see https://github.com/llvm/llvm-project/pull/134196#discussion_r2068617059


Compare: https://github.com/llvm/llvm-project/compare/e87aaaf3274d...e66eb836e67f

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications