[PATCH] Allow machine hierarchy expansion

hfinkel at anl.gov hfinkel at anl.gov
Sat May 23 17:56:57 PDT 2015


I don't understand this patch; can you please explain more?

Is the idea that if, at runtime, someone adds more CPUs, the hierarchy will resize itself?


REPOSITORY
  rL LLVM

================
Comment at: runtime/src/kmp_affinity.cpp:320
@@ -319,3 +319,3 @@
         etc.  We don't want to get specific with nomenclature */
-    static const kmp_uint32 maxLevels=7;
+    kmp_uint32 maxLevels;
 
----------------
Why are you making this dynamic? How could this end up needing more than 7?

http://reviews.llvm.org/D9900

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list