Skip to content

Commit

Permalink
Merge branch 'develop' into bindir
Browse files Browse the repository at this point in the history
  • Loading branch information
balos1 authored May 15, 2024
2 parents 4767385 + e8b7acd commit f21d80f
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 3 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,9 @@ Fixed a CMake bug that caused an MPI linking error for our C++ examples in some
Fixed a bug in `ARKodeSPRKTable_Create` where the coefficient arrays where not
allocated.

Fix bug on LLP64 platforms (like Windows 64-bit) where `KLU_INDEXTYPE` could be
32 bits wide even if `SUNDIALS_INT64_T` is defined.

## Changes to SUNDIALS in release v7.0.0

### Major Feature
Expand Down
4 changes: 2 additions & 2 deletions cmake/tpl/FindKLU.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ if (NOT (KLU_INCLUDE_DIR OR KLU_LIBRARY_DIR OR KLU_LIBRARY))
endif()

# Set library prefixes for Windows
if(WIN32)
if(MSVC OR ("${CMAKE_C_SIMULATE_ID}" STREQUAL "MSVC"))
set(CMAKE_FIND_LIBRARY_PREFIXES lib ${CMAKE_FIND_LIBRARY_PREFIXES})
set(CMAKE_FIND_LIBRARY_SUFFIXES d.lib ${CMAKE_FIND_LIBRARY_SUFFIXES})
elseif(APPLE)
Expand Down Expand Up @@ -95,7 +95,7 @@ endif ()
if (NOT SUITESPARSECONFIG_LIBRARY)
set(SUITESPARSECONFIG_LIBRARY_NAME suitesparseconfig)
# NOTE: no prefix for this library on windows
if(WIN32 AND NOT MSYS)
if(MSVC OR ("${CMAKE_C_SIMULATE_ID}" STREQUAL "MSVC"))
set(CMAKE_FIND_LIBRARY_PREFIXES "")
endif()
find_library( SUITESPARSECONFIG_LIBRARY ${SUITESPARSECONFIG_LIBRARY_NAME} ${KLU_LIBRARY_DIR} NO_DEFAULT_PATH)
Expand Down
3 changes: 3 additions & 0 deletions doc/shared/RecentChanges.rst
Original file line number Diff line number Diff line change
Expand Up @@ -69,3 +69,6 @@ Fixed a CMake bug that caused an MPI linking error for our C++ examples in some

Fixed a bug in :c:func:`ARKodeSPRKTable_Create` where the coefficient arrays
where not allocated.

Fix bug on LLP64 platforms (like Windows 64-bit) where ``KLU_INDEXTYPE`` could be
32 bits wide even if ``SUNDIALS_INT64_T`` is defined.
3 changes: 2 additions & 1 deletion src/sunlinsol/klu/sunlinsol_klu.c
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
* the SUNLINSOL package.
* -----------------------------------------------------------------*/

#include <stdint.h>
#include <stdio.h>
#include <stdlib.h>
#include <sundials/sundials_math.h>
Expand Down Expand Up @@ -49,7 +50,7 @@
*/

#if defined(SUNDIALS_INT64_T)
#define KLU_INDEXTYPE long int
#define KLU_INDEXTYPE int64_t
#else
#define KLU_INDEXTYPE int
#endif
Expand Down

0 comments on commit f21d80f

Please sign in to comment.