From ec9962d7dbd87d7e5f6af9a68a666378ea3470f6 Mon Sep 17 00:00:00 2001
From: Cosmin Stejerean <cstejerean@fb.com>
Date: Thu, 18 Jan 2024 16:11:45 -0800
Subject: [PATCH] update pip install instructions for macOS to use homebrew
 llvm

---
 resource/doc/python.md | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/resource/doc/python.md b/resource/doc/python.md
index 683b05d57..e0a7c6b48 100644
--- a/resource/doc/python.md
+++ b/resource/doc/python.md
@@ -41,7 +41,7 @@ brew install python3
 Install the remaining dependencies:
 
 ```bash
-brew install nasm doxygen
+brew install nasm doxygen llvm
 ```
 
 Note that `brew` requires no `sudo`.
@@ -76,6 +76,13 @@ Install the rest of the required Python packages:
 pip3 install -r python/requirements.txt
 ```
 
+On macOS it's important to use the LLVM from homebrew as the macOS clang does not include support for OpenMP, which is needed for libsvm-official
+
+``` shell script
+LLVM_CONFIG=$HOMEBREW_PREFIX/opt/llvm/bin/llvm-config pip install -r ./python/requirements.txt
+```
+
+
 ## Testing
 
 Run unittests and make sure they all pass: