Make pybind rely on main proj

Author: haochengxia

.github/workflows/python.yml

@@ -21,11 +21,18 @@ jobs:
           pip install --upgrade pip
           pip install -r requirements.txt
           pip install pytest
+          pip install pybind11
+          pip install scikit-build
       
       - name: Build libCacheSim-python
         run: |
+          cmake -B ${{github.workspace}}/build \
+                -DCMAKE_BUILD_TYPE=${{env.BUILD_TYPE}} \
+                -DCMAKE_C_FLAGS="-fPIC" \
+                -DCMAKE_CXX_FLAGS="-fPIC"
+          cmake --build ${{github.workspace}}/build --config ${{env.BUILD_TYPE}}
           cd libCacheSim-python
-          pip install -e .
+          pip install -e . --no-build-isolation
       
       - name: Run tests
         run: |

CMakeLists.txt

@@ -25,6 +25,9 @@ option(SUPPORT_TTL "whether support TTL" OFF)
 option(OPT_SUPPORT_ZSTD_TRACE "whether support zstd trace" ON)
 option(ENABLE_LRB "enable LRB" OFF)
 option(ENABLE_3L_CACHE "enable 3LCache" OFF)
+# For python binding
+option(BUILD_PYTHON_BINDING "Build Python binding" OFF)
+option(ENABLE_PIC "Build all targets with -fPIC (for shared libs/Python bindings)" OFF)
 set(LOG_LEVEL "default" CACHE STRING "change the logging level")
 set_property(CACHE LOG_LEVEL PROPERTY STRINGS ERROR WARN INFO DEBUG VERBOSE DEFAULT)
 
@@ -41,6 +44,11 @@ else()
     message(FATAL_ERROR "unsupported operating system ${CMAKE_SYSTEM_NAME}")
 endif()
 
+if(ENABLE_PIC)
+    message(STATUS "enabling position independent code")
+    set(CMAKE_POSITION_INDEPENDENT_CODE ON)
+endif()
+
 configure_file(libCacheSim/include/config.h.in libCacheSim/include/config.h)
 
 if(SUPPORT_TTL)
@@ -295,3 +303,10 @@ install(EXPORT ${PROJECT_NAME}Targets
     DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/${PROJECT_NAME}
 )
 
+# Python binding subdir
+if(BUILD_PYTHON_BINDING)
+    message(STATUS "Building Python binding")
+    add_subdirectory(${CMAKE_CURRENT_SOURCE_DIR}/libCacheSim-python)
+else()
+    message(STATUS "Skipping Python binding (use -DBUILD_PYTHON_BINDING=ON to enable)")
+endif()