BUILD: Change default buildtype from debug to release

.gitlab/build.sh

@@ -227,7 +227,7 @@ export CMAKE_PREFIX_PATH="${INSTALL_DIR}:${CMAKE_PREFIX_PATH}"
 export UCX_TLS=^cuda_ipc
 
 # shellcheck disable=SC2086
-meson setup nixl_build --prefix=${INSTALL_DIR} -Ducx_path=${UCX_INSTALL_DIR} -Dbuild_docs=true -Drust=false ${EXTRA_BUILD_ARGS} -Dlibfabric_path="${LIBFABRIC_INSTALL_DIR}"
+meson setup nixl_build --prefix=${INSTALL_DIR} -Ducx_path=${UCX_INSTALL_DIR} -Dbuild_docs=true -Drust=false ${EXTRA_BUILD_ARGS} -Dlibfabric_path="${LIBFABRIC_INSTALL_DIR}" --buildtype=debug
 ninja -j"$NPROC" -C nixl_build && ninja -j"$NPROC" -C nixl_build install
 mkdir -p dist && cp nixl_build/src/bindings/python/nixl-meta/nixl-*.whl dist/
 

README.md

@@ -102,16 +102,16 @@ $ ninja install
 
 ### Build Options
 
-#### Release build
+#### Release build (default)
 
 ```bash
 $ meson setup <name_of_build_dir> --buildtype=release
 ```
 
-#### Debug build (default)
+#### Debug build
 
 ```bash
-$ meson setup <name_of_build_dir>
+$ meson setup <name_of_build_dir> --buildtype=debug
 ```
 
 #### NIXL-specific build options
@@ -170,7 +170,7 @@ For Python examples, see [examples/python/](examples/python/).
 ### Rust Bindings
 #### Build
 - Use `-Drust=true` meson option to build rust bindings.
-- Use `-Ddebug=false` for a release build.
+- Use `--buildtype=debug` for a debug build (default is release).
 - Or build manually:
     ```bash
     $ cargo build --release

meson.build

@@ -14,7 +14,7 @@
 # limitations under the License.
 
 project('nixl', 'CPP', version: '0.7.1',
-    default_options: ['buildtype=debug',
+    default_options: ['buildtype=release',
                 'werror=true',
                 'cpp_std=c++17',
                 'prefix=/opt/nvidia/nvda_nixl'],

src/plugins/gpunetio/README.md

@@ -51,5 +51,5 @@ NIXL bench also has the option to specify the DOCA GPUNetIO backend. An example
 
 ## Caveats
 
-By default NIXL is built with `buildtype=debug` option. This is ok for correctness and debugging.
-To run for performace (e.g. with NIXL bench) t's hightly recommended to build NIXL with `buildtype=release`.
+To run for performance (e.g. with NIXL bench) it's highly recommended to build NIXL with `buildtype=release`.
+For debugging purposes, NIXL can be built with `buildtype=debug`.