[PATCH] D156193: Fix thinLTO long compile time problem

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added inline comments.


================
Comment at: llvm/lib/Support/Unix/Threading.inc:386
 }
 #elif defined(__linux__) && !defined(__ANDROID__)
 static int computeHostNumPhysicalCores() {
----------------
Are you compiling for Android? If not, it should have used this case, which does a superset of what you have added above.  I tracked the Android exclusion down to this commit: e9b213131ae9c57f4f151d3206916676135b31b0, which says:

```
Android is excluded because of the higher API level requirement:
`sched_getaffinity; # introduced-arm=12 introduced-arm64=21 introduced-x86=12 introduced-x86_64=21`
```

Is that an issue then for aarch64 which iiuc is arm64? Or should the code be restructured so aarch64 falls into this case for Android?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156193