[all-commits] [llvm/llvm-project] cd2412: [clang] Remove a FIXME that we can't fix

Nico Weber via All-commits all-commits at lists.llvm.org
Tue Aug 23 07:13:18 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: cd24120c9d9544b202641fbff46be38175bb2470
      https://github.com/llvm/llvm-project/commit/cd24120c9d9544b202641fbff46be38175bb2470
  Author: Nico Weber <thakis at chromium.org>
  Date:   2022-08-23 (Tue, 23 Aug 2022)

  Changed paths:
    M clang/lib/Driver/ToolChains/Darwin.cpp

  Log Message:
  -----------
  [clang] Remove a FIXME that we can't fix

I added this recently, but it looks like several tests very intentionally
check that `-mios-version-min=foo --target=x86_64-apple-ios` does simulator
builds. So we can't easily remove this hack, even though it makes little
sense in an arm mac world. (Here, you _have_ to say
`-mios-simulator-version-min=` or `--target=arm64-apple-ios-simulator`.)

The tests that check this:
  Clang :: Driver/darwin-ld.c
  Clang :: Driver/darwin-simulator-macro.c
  Clang :: Driver/darwin-version.c

No behavior change.

Differential Revision: https://reviews.llvm.org/D132400




More information about the All-commits mailing list