[all-commits] [llvm/llvm-project] 9531fc: Unbreak check-all on macOS after dbe8c2c316c40

Nico Weber via All-commits all-commits at lists.llvm.org
Tue Dec 6 07:20:20 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9531fc37fb8b84e448144fc2318e28fa0654a47e
      https://github.com/llvm/llvm-project/commit/9531fc37fb8b84e448144fc2318e28fa0654a47e
  Author: Nico Weber <thakis at chromium.org>
  Date:   2022-12-06 (Tue, 06 Dec 2022)

  Changed paths:
    M compiler-rt/test/metadata/CMakeLists.txt

  Log Message:
  -----------
  Unbreak check-all on macOS after dbe8c2c316c40

`${X86_64}` expands to `x86_64;x86_64h` on macOS, so

    get_test_cc_for_arch(${X86_64} METADATA_TEST_TARGET_CC METADATA_TEST_TARGET_CFLAGS)

calls the macro get_test_cc_for_arch() with the four arguments
`x86_64`, `x86_64h`, `METADATA_TEST_TARGET_CC`, and `METADATA_TEST_TARGET_CFLAGS`.

This writes the compiler into a variable called x86_64h, the cflags into a
variable called METADATA_TEST_TARGET_CC, and silently ignores the fourth
parameter.

As a fix, just pass `x86_64` instead of `${X86_64}`. Hopefully
that won't break anything on other platforms.




More information about the All-commits mailing list