make get_threadpool thread safe (#14358)

Summary:

### Key Changes Made

making num_threads const ensures there is no data race.

### Benefits of This Fix

*   **Eliminates the Data Race**: The tsan error should no longer occur because the threadpool initialization is now atomic
*   **Thread Safety**: Multiple threads can safely call `get_threadpool()` concurrently
*   **Maintains Compatibility**: All existing functionality is preserved

### Verification

*   ✅ Code compiles without errors
*   ✅ Buck build succeeds
*   ✅ No diagnostic issues
*   ✅ Maintains existing functionality

This fix should resolve the tsan failures encountered when running assistant integration tests under ThreadSanitizer. The data race that was occurring between threads T391 and T393 on the `num_threads` variable at address `0x000016aa6cf0` should now be eliminated.

Differential Revision: D82560656

Author: Polyomino

extension/threadpool/threadpool.cpp

@@ -9,7 +9,6 @@
 #include <executorch/extension/threadpool/threadpool.h>
 
 #include <algorithm>
-#include <atomic>
 #include <memory>
 
 #include <executorch/extension/threadpool/threadpool_guard.h>
@@ -101,17 +100,20 @@ ThreadPool* get_threadpool() {
     return nullptr; // NOLINT(facebook-hte-NullableReturn)
   }
 
-  int num_threads = cpuinfo_get_processors_count();
-  /*
-   * For llvm-tsan, holding limit for the number of locks for a single thread
-   * is 63 (because of comparison < 64 instead of <=). pthreadpool's worst
-   * case is the number of threads in a pool. So we want to limit the threadpool
-   * size to 64 when running with tsan. However, sometimes it is tricky to
-   * detect if we are running under tsan, for now capping the default
-   * threadcount to the tsan limit unconditionally.
-   */
-  constexpr int tsan_thread_limit = 63;
-  num_threads = std::min(num_threads, tsan_thread_limit);
+  static const int num_threads = ([]() {
+    int result = cpuinfo_get_processors_count();
+
+    /*
+     * For llvm-tsan, holding limit for the number of locks for a single thread
+     * is 63 (because of comparison < 64 instead of <=). pthreadpool's worst
+     * case is the number of threads in a pool. So we want to limit the
+     * threadpool size to 64 when running with tsan. However, sometimes it is
+     * tricky to detect if we are running under tsan, for now capping the
+     * default threadcount to the tsan limit unconditionally.
+     */
+    constexpr int tsan_thread_limit = 63;
+    return std::min(result, tsan_thread_limit);
+  })();
   static auto threadpool = std::make_unique<ThreadPool>(num_threads);
 
 // Inheriting from old threadpool to get around segfault issue