Basic C API for GauXC by awvwgk · Pull Request #172 · wavefunction91/GauXC

awvwgk · 2026-01-19T12:30:49Z

Closes #171

Copilot

Pull request overview

This pull request adds a comprehensive C API for the GauXC library, enabling integration with C codebases and other languages that interface with C. The implementation wraps core GauXC C++ functionality including molecular structures, basis sets, integration grids, load balancing, and XC integrators.

Changes:

Added C API bindings for core classes (Molecule, Atom, BasisSet, Shell, etc.)
Implemented factory patterns for runtime environments, load balancers, molecular weights, and XC integrators
Created C/C++ compatible enum definitions with mappings between C and C++ versions
Added HDF5 I/O support for C API
Refactored configuration headers to separate C and C++ concerns

Reviewed changes

Copilot reviewed 47 out of 47 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
include/gauxc/*.h	New C API header files defining opaque handles and functions for core types
include/gauxc/util/c_*.hpp	Helper headers for casting between C handles and C++ objects
src/c_*.cxx	C API implementation files wrapping C++ functionality
include/gauxc/enum.h	C-compatible enum definitions
include/gauxc/enums.hpp	C++ enum class definitions mapped to C enums
include/gauxc/gauxc_config.h.in	C-compatible configuration header
include/gauxc/gauxc_config.hpp	C++ configuration header including C header
tests/moltypes_test.cxx	Basic C API interoperability test
src/CMakeLists.txt	CMake integration for conditional C API compilation
src/external/c_hdf5_*.cxx	HDF5 I/O functions for C API

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

src/c_xc_integrator.cxx

include/gauxc/load_balancer.h

src/c_molecule.cxx

src/external/c_hdf5_write.cxx

include/gauxc/molecule.h

tests/moltypes_test.cxx

include/gauxc/xc_integrator.h

src/c_functional.cxx

src/c_xc_integrator.cxx

Copilot

Pull request overview

Copilot reviewed 48 out of 48 changed files in this pull request and generated 17 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

include/gauxc/xc_integrator.h

src/c_molecule.cxx

include/gauxc/external/hdf5.h

src/CMakeLists.txt

src/c_molgrid.cxx

include/gauxc/util/c_matrix.hpp

include/gauxc/util/c_basisset.hpp

src/c_matrix.cxx

src/c_functional.cxx

include/gauxc/types.h

- provide C enums - add atom and molecule types - add basis set and shell definitions - add molecule grid and runtime environment - add load balancer to C API - add molecular weights for C API - add functional class wrapping ExchCXX - add xc integrator and matrix type - add references for functionals - add support for reading and writing from HDF5 in C

wavefunction91 · 2026-01-30T23:02:00Z

include/gauxc/external/hdf5.h

@@ -0,0 +1,112 @@
+/**


Global comment: I'd prefer that C-related things (and similarly for #174) are placed in gauxc/c_api and gauxc/fortran_api respectively. The top-level is messy enough as it is.

wavefunction91 · 2026-01-30T23:04:49Z

include/gauxc/util/c_basisset.hpp

@@ -0,0 +1,47 @@
+/**


Similarly, this seems to be glue to stitch together the C/C++ APIs? I'd combine all of these types of utilities into either a single file, or into a single directory.

wavefunction91 · 2026-01-30T23:05:20Z

include/gauxc/util/c_basisset.hpp

+static inline BasisSet<double>* get_basisset_ptr(C::GauXCBasisSet basis) noexcept {
+  return static_cast<BasisSet<double>*>(basis.ptr);
+}
+static inline Shell<double> convert_shell(C::GauXCShell shell, bool normalize) noexcept {


Why is this not namespaced? This can't be called from C.

wavefunction91 · 2026-01-30T23:05:47Z

include/gauxc/util/c_basisset.hpp

+static inline BasisSet<double>* get_basisset_ptr(C::GauXCBasisSet basis) noexcept {
+  return static_cast<BasisSet<double>*>(basis.ptr);
+}
+static inline Shell<double> convert_shell(C::GauXCShell shell, bool normalize) noexcept {


Unless the functions are very small, I'd prefer that they get compiled into the library rather than living in a header.

wavefunction91 · 2026-01-30T23:40:15Z

include/gauxc/util/c_status.hpp

+
+namespace GauXC::detail {
+
+static inline char* strdup(const char* str) {


These kinds of functions make me nervous - is there a reason we don't just use the POSIX/C23 version of this? Also, why is this in the status header?

wavefunction91 · 2026-01-30T23:43:05Z

include/gauxc/util/mpi.h

+#pragma once
+#include <gauxc/gauxc_config.h>
+
+#ifdef GAUXC_HAS_MPI


I'd prefer that we do not duplicate this kind of code.

wavefunction91 · 2026-01-30T23:44:15Z

include/gauxc/basisset.h

+ */
+#pragma once
+
+#ifdef __cplusplus


This kind of include scheme makes me very nervous - are the functions in these headers guaranteed to have the same ABI?

wavefunction91 · 2026-01-30T23:47:12Z

include/gauxc/functional.h

+#endif
+
+enum GauXC_Functional {
+  /// @brief Slater exchange & Vosko, Wilk & Nusair correlation (VWN3)


I think we talked about this in this PR wavefunction91/ExchCXX#49, and I'd prefer that we remain consistent.

wavefunction91 · 2026-01-30T23:49:31Z

include/gauxc/gauxc_config.h.in

@@ -31,10 +32,4 @@
 #ifdef GAUXC_HAS_DEVICE
 #cmakedefine GAUXC_GPU_XC_MAX_AM     @GAUXC_GPU_XC_MAX_AM@
 #cmakedefine GAUXC_GPU_SNLINK_MAX_AM @GAUXC_GPU_SNLINK_MAX_AM@
-#endif
-
-#if defined(__CUDACC__) || defined(__HIPCC__)


Why can't this just stay here?

wavefunction91 · 2026-01-30T23:58:21Z

include/gauxc/util/c_matrix.hpp

@@ -0,0 +1,162 @@
+/**


Global comment on use of managed matrix types in C - not a fan. I'd much prefer that we delegate all memory management of matrices to the calling program. In fact, we already do that under the hood. When I had considered writing a C-API in the past, my intention was always to expose the raw pointer API to the user and let them handle it (in a BLAS/LAPACK style convention). Do you have thoughts on this @susilehtola - I just remember the C-API being a point you had brought up several times.

Copilot

Pull request overview

Copilot reviewed 49 out of 49 changed files in this pull request and generated 10 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-06T23:02:21Z

src/c_xc_integrator.cxx

+void gauxc_integrator_eval_exc_vxc_rks(
+  GauXCStatus* status,
+  const GauXCIntegrator integrator,
+  const GauXCMatrix density_matrix,
+  double* exc_out,
+  GauXCMatrix* vxc_matrix
+) {
+  vxc_matrix->ptr = nullptr;
+
+  try {
+    auto& dm = *detail::get_matrix_ptr(density_matrix);
+
+    auto [exc, vxc] = integrator.owned
+      ? detail::get_xc_integrator_ptr(integrator)->eval_exc_vxc(dm)
+      : detail::get_xc_integrator_shared(integrator)->get()->eval_exc_vxc(dm);
+
+    *exc_out = exc;
+    vxc_matrix->ptr = new detail::CMatrix(std::move(vxc));
+    status->code = 0;


The output GauXCMatrix* vxc_matrix handle is only partially populated: hdr.type is never set, and any pre-existing vxc_matrix->ptr is overwritten without being freed (memory leak). Initialize vxc_matrix->hdr to GauXC_Type_Matrix and either require/verify vxc_matrix is empty or delete/reuse any existing matrix storage before assigning a new pointer.

Copilot · 2026-02-06T23:02:21Z

src/c_xc_integrator.cxx

+void gauxc_integrator_eval_exc_vxc_gks(
+  GauXCStatus* status,
+  const GauXCIntegrator integrator,
+  const GauXCMatrix density_matrix_s,
+  const GauXCMatrix density_matrix_z,
+  const GauXCMatrix density_matrix_x,
+  const GauXCMatrix density_matrix_y,
+  double* exc_out,
+  GauXCMatrix* vxc_matrix_s,
+  GauXCMatrix* vxc_matrix_z,
+  GauXCMatrix* vxc_matrix_x,
+  GauXCMatrix* vxc_matrix_y
+) {
+  vxc_matrix_s->ptr = nullptr;
+  vxc_matrix_z->ptr = nullptr;
+  vxc_matrix_x->ptr = nullptr;
+  vxc_matrix_y->ptr = nullptr;
+
+  try {
+    auto& dm_s = *detail::get_matrix_ptr(density_matrix_s);
+    auto& dm_z = *detail::get_matrix_ptr(density_matrix_z);
+    auto& dm_x = *detail::get_matrix_ptr(density_matrix_x);
+    auto& dm_y = *detail::get_matrix_ptr(density_matrix_y);
+
+    auto [exc, vxc_s, vxc_z, vxc_x, vxc_y] = integrator.owned
+      ? detail::get_xc_integrator_ptr(integrator)->eval_exc_vxc(dm_s, dm_z, dm_x, dm_y)
+      : detail::get_xc_integrator_shared(integrator)->get()->eval_exc_vxc(dm_s, dm_z, dm_x, dm_y);
+
+    *exc_out = exc;
+    vxc_matrix_s->ptr = new detail::CMatrix(std::move(vxc_s));
+    vxc_matrix_z->ptr = new detail::CMatrix(std::move(vxc_z));
+    vxc_matrix_x->ptr = new detail::CMatrix(std::move(vxc_x));
+    vxc_matrix_y->ptr = new detail::CMatrix(std::move(vxc_y));
+    status->code = 0;


Same as other eval_exc_vxc_* APIs: all vxc_matrix_* outputs should have hdr.type set and should not leak by overwriting an existing ptr. Consider either (1) filling preallocated matrices (resize + write) or (2) explicitly deleting any existing ptr before assigning a new allocation and documenting ownership clearly.

Copilot · 2026-02-06T23:02:21Z

include/gauxc/status.h

+typedef struct GauXCStatus {
+  int code;
+  char* message;
+} GauXCStatus;


GauXCStatus has a raw char* message, but there’s no documented/implemented initialization/reset contract. Because most C API entry points only set status->code (not status->message), C++ callers that default-initialize GauXCStatus can end up with an indeterminate message pointer and crash on gauxc_status_delete. Consider (a) documenting that callers must zero-initialize ({0,nullptr}), and (b) having every API entry point clear/free any previous message and set message=nullptr on success.

Copilot · 2026-02-06T23:02:22Z

tests/moltypes_test.cxx

+    C::GauXCStatus status;
+    C::GauXCMolecule mol = C::gauxc_molecule_new(&status);
+    CHECK(status.code == 0);


GauXCStatus is a POD and C::GauXCStatus status; leaves status.message uninitialized in C++. Since the C API doesn’t consistently null out/clear status.message on success, later calls like gauxc_status_delete(&status) could delete[] a garbage pointer. Prefer zero-initializing here (e.g., {} / {0,nullptr}) and/or ensure the C API resets status.message on entry/success.

Copilot · 2026-02-06T23:02:22Z

src/c_matrix.cxx

+GauXCMatrix gauxc_matrix_empty(
+  GauXCStatus* status
+) {
+  status->code = 0;
+  GauXCMatrix matrix;
+  matrix.hdr = GauXCHeader{GauXC_Type_Matrix};
+  matrix.ptr = new detail::CMatrix();
+


gauxc_matrix_empty performs new detail::CMatrix() without a try/catch. If allocation throws (e.g., std::bad_alloc), the exception will cross the C API boundary. Wrap the allocation like gauxc_matrix_new does and set status->code/message + return {.ptr=nullptr} on failure.

Copilot · 2026-02-06T23:02:22Z

src/c_functional.cxx

+    auto polar = polarized ? ExchCXX::Spin::Polarized : ExchCXX::Spin::Unpolarized;
+    functional_type func;
+    if(ExchCXX::functional_map.key_exists(functional_spec)) {
+      func = functional_type( ExchCXX::Backend::builtin, ExchCXX::functional_map.value(functional_spec), 
+        polar );
+    } 
+#ifdef EXCHCXX_ENABLE_LIBXC
+    else { 
+      std::vector<std::pair<double, ExchCXX::XCKernel>> funcs;
+      std::vector<std::string> libxc_names;
+      detail::split(libxc_names, functional_spec, ",");
+      for( auto n : libxc_names ) {
+        funcs.push_back( {1.0, ExchCXX::XCKernel(ExchCXX::libxc_name_string(n), polar)} );
+      }
+      func = functional_type(funcs);
+    }
+#endif
+
+    functional.ptr = new functional_type( std::move(func) );
+  } catch (std::exception& e) {


If functional_spec is not in ExchCXX::functional_map and EXCHCXX_ENABLE_LIBXC is not enabled, func is left without being assigned from the input spec, yet the code still constructs and returns a functional_type from it. This should instead report an error (set nonzero status / throw) when the functional string is unknown and libxc support isn’t available.

Copilot · 2026-02-06T23:02:22Z

src/c_xc_integrator.cxx

+void gauxc_integrator_eval_exc_vxc_uks(
+  GauXCStatus* status,
+  const GauXCIntegrator integrator,
+  const GauXCMatrix density_matrix_s,
+  const GauXCMatrix density_matrix_z,
+  double* exc_out,
+  GauXCMatrix* vxc_matrix_s,
+  GauXCMatrix* vxc_matrix_z
+) {
+  vxc_matrix_s->ptr = nullptr;
+  vxc_matrix_z->ptr = nullptr;
+
+  try {
+    auto& dm_s = *detail::get_matrix_ptr(density_matrix_s);
+    auto& dm_z = *detail::get_matrix_ptr(density_matrix_z);
+
+    auto [exc, vxc_s, vxc_z] = integrator.owned
+      ? detail::get_xc_integrator_ptr(integrator)->eval_exc_vxc(dm_s, dm_z)
+      : detail::get_xc_integrator_shared(integrator)->get()->eval_exc_vxc(dm_s, dm_z);
+
+    *exc_out = exc;
+    vxc_matrix_s->ptr = new detail::CMatrix(std::move(vxc_s));
+    vxc_matrix_z->ptr = new detail::CMatrix(std::move(vxc_z));
+    status->code = 0;


Same as RKS: vxc_matrix_s/vxc_matrix_z output handles don’t have hdr.type initialized and their ptr fields are overwritten without freeing any existing allocation. This can break gauxc_object_delete (type dispatch) and can leak memory if the caller reuses output matrices.

Copilot · 2026-02-06T23:02:23Z

tests/moltypes_test.cxx

+#ifdef GAUXC_HAS_C
+  SECTION("C HDF5 IO") {
+    C::GauXCStatus status;


The "C HDF5 IO" section is only guarded by GAUXC_HAS_C, but the declarations in <gauxc/external/hdf5.h> are only available when GAUXC_HAS_HDF5 is set. If C API is enabled but HDF5 is disabled, this test will fail to compile. Guard this section (and/or the include) with GAUXC_HAS_HDF5 as well.

Copilot · 2026-02-06T23:02:23Z

tests/moltypes_test.cxx

+    C::GauXCStatus status;
+


Same issue as earlier: C::GauXCStatus status; does not initialize status.message in C++. Use value-initialization ({}) to avoid leaving message as an indeterminate pointer (which can later crash on gauxc_status_delete).

Copilot · 2026-02-06T23:02:23Z

src/c_types.cxx

+void gauxc_object_delete(GauXCStatus* status, void** obj) {
+   status->code = 0;
+   if(obj == nullptr) return;
+
+   struct GauXCObject {
+      GauXCHeader hdr;
+   };
+
+   GauXCHeader* header = &reinterpret_cast<struct GauXCObject*>(*obj)->hdr;
+


gauxc_object_delete dereferences *obj unconditionally (reinterpret_cast<...>(*obj)), but only checks obj == nullptr. If the caller passes a null object pointer (common when deleting optional handles), this will crash. Add a *obj == nullptr guard before reading the header, and consider nulling *obj after a successful delete since the API takes a void**.

awvwgk self-assigned this Jan 19, 2026

awvwgk force-pushed the c-api branch 3 times, most recently from dd396c2 to 5881a76 Compare January 26, 2026 09:54

awvwgk mentioned this pull request Jan 26, 2026

Add guide for using GauXC from C and Fortran microsoft/skala#23

Open

awvwgk force-pushed the c-api branch 3 times, most recently from 1d61b38 to 83841ad Compare January 27, 2026 10:05

awvwgk changed the title ~~[WIP] Basic C API for GauXC~~ Basic C API for GauXC Jan 27, 2026

awvwgk requested review from Copilot and wavefunction91 January 27, 2026 10:59

awvwgk added the enhancement New feature or request label Jan 27, 2026

Copilot started reviewing on behalf of awvwgk January 27, 2026 11:00 View session

Copilot AI reviewed Jan 27, 2026

View reviewed changes

awvwgk force-pushed the c-api branch from f006bbd to 16abe8f Compare January 28, 2026 13:43

awvwgk added the C API Related to the C API label Jan 28, 2026

awvwgk requested a review from Copilot January 29, 2026 07:46

Copilot started reviewing on behalf of awvwgk January 29, 2026 07:46 View session

Copilot AI reviewed Jan 29, 2026

View reviewed changes

awvwgk force-pushed the c-api branch from 16abe8f to 1c87794 Compare January 30, 2026 13:36

Fix check for number of primitives

ff16199

awvwgk force-pushed the c-api branch from cf0bb8a to ff16199 Compare February 6, 2026 14:24

wavefunction91 requested changes Feb 6, 2026

View reviewed changes

wavefunction91 requested a review from Copilot February 6, 2026 22:55

Copilot started reviewing on behalf of wavefunction91 February 6, 2026 22:55 View session

Copilot AI reviewed Feb 6, 2026

View reviewed changes


		namespace GauXC::detail {

		static inline char* strdup(const char* str) {

Conversation

awvwgk commented Jan 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

awvwgk commented Jan 19, 2026 •

edited

Loading