[Openmp-commits] [PATCH] D93169: [OpenMP] Added the support for cache line size 256 for A64FX

[Chandler Carruth via Phabricator via Openmp-commits]

chandlerc added a comment.

I'm going to revert this as it breaks CMake on systems which do not have `/proc/cpuinfo` such as macOS.

This may be a bit hard to see because the code isn't reached unless the architecture is `aarch64`, but on an ARM macOS system that path hits. It would also hit on other BSDs or other OSes running on AArch64 but without `/proc/cpuinfo`.

For your reference, here is the error message from CMake for me:

  CMake Error at /Users/chandlerc/src/llvm/llvm-project/openmp/runtime/cmake/LibompGetArchitecture.cmake:74 (file):
    file failed to open for reading (No such file or directory):
  
      /proc/cpuinfo
  Call Stack (most recent call first):
    /Users/chandlerc/src/llvm/llvm-project/openmp/runtime/CMakeLists.txt:73 (libomp_is_aarch64_a64fx)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93169