[Openmp-commits] [PATCH] D41000: AArch64: fix cpuinfo issues

[Paul Osmialowski via Phabricator via Openmp-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL320593: [AArch64] fix an issue with older /proc/cpuinfo layout (authored by pawosm01, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D41000?vs=126279&id=126765#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D41000

Files:
  openmp/trunk/runtime/src/kmp_affinity.cpp


Index: openmp/trunk/runtime/src/kmp_affinity.cpp
===================================================================
--- openmp/trunk/runtime/src/kmp_affinity.cpp
+++ openmp/trunk/runtime/src/kmp_affinity.cpp
@@ -2028,7 +2028,15 @@
         if ((p == NULL) || (KMP_SSCANF(p + 1, "%u\n", &val) != 1))
           goto no_val;
         if (threadInfo[num_avail][osIdIndex] != UINT_MAX)
+#if KMP_ARCH_AARCH64
+          // Handle the old AArch64 /proc/cpuinfo layout differently,
+          // it contains all of the 'processor' entries listed in a
+          // single 'Processor' section, therefore the normal looking
+          // for duplicates in that section will always fail.
+          num_avail++;
+#else
           goto dup_field;
+#endif
         threadInfo[num_avail][osIdIndex] = val;
 #if KMP_OS_LINUX && !(KMP_ARCH_X86 || KMP_ARCH_X86_64)
         char path[256];


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41000.126765.patch
Type: text/x-patch
Size: 875 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20171213/1aa30540/attachment.bin>