[PATCH] D101855: [lld-macho] Check simulator platforms to avoid issuing false positive errors.

Jez Ng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 4 14:53:26 PDT 2021


int3 added a comment.

this needs a test :)



================
Comment at: lld/MachO/InputFiles.cpp:144
+  // Mapping of platform to simulator and vice-versa.
+  static std::map<PlatformKind, PlatformKind> *platformMap = []() {
+    auto *ret = new std::map<PlatformKind, PlatformKind>();
----------------
nit 1: for uniformity, can we use a DenseMap instead of a std::map? I know perf isn't important here but it would be nice to avoid std::map where possible

nit 2: I'm pretty sure we can initialize this via initializer lists rather than a lambda, i.e. something like

```
platformMap = {{PlatformKind::iOS, PlatformKind::iOSSimulator}, {PlatformKind::iOS, PlatformKind::iOSSimulator}, ...};
```


================
Comment at: lld/MachO/InputFiles.cpp:167
     return true;
   // TODO: Correctly detect simulator platforms or relax this check.
+  if (config->platform() != platformInfo->target.Platform &&
----------------
rm?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101855/new/

https://reviews.llvm.org/D101855



More information about the llvm-commits mailing list