Skip to content

Commit

Permalink
Build is failing with configure options enable-multithreading, enabl…
Browse files Browse the repository at this point in the history
…e-supermatrix

 1. Fixed LIBFLAME_IMPORT error while enabling multithreading and supermatrix
 2. Fixed SORHR_COL  symbol issue from Windows build.

AMD Internal : [CPUPL-1661]

Change-Id: I82f1d9d736899e9e2661445674e8f480e516ff57
  • Loading branch information
nphaniku committed Jun 22, 2021
1 parent 5b7b8c0 commit 593e464
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 5 deletions.
8 changes: 5 additions & 3 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -46,10 +46,9 @@ option (ENABLE_AUTODETECT_F77_UNDERSCORING "Enable autodetect F77 Underscore" OF
option (ENABLE_UPPERCASE_BLAS "Enable invocation of Blas with Flame" OFF)
option (ENABLE_UPPERCASE_LAPACK "Enable invocation of Lapack with Flame" OFF)
option (ENABLE_GPU "Enable gpu advantage" OFF)

option (ENABLE_XBLAS "Enable xblas API's" OFF)

set (ENABLE_WINDOWS_BUILD "Enables windows build" ON)
add_compile_definitions(FLA_ENABLE_WINDOWS_BUILD=1)

if(ENABLE_UPPERCASE)
add_definitions(-DBLIS1_ENABLE_UPPERCASE_F77)
Expand All @@ -58,6 +57,9 @@ elseif(NOT ENABLE_UPPERCASE)
add_definitions(-DNOCHANGE)
endif()

if(ENABLE_XBLAS)
add_definitions(-DFLA_ENABLE_XBLAS)
endif()
if (ENABLE_NON_CRITICAL_CODE)
set (FLA_ENABLE_NON_CRITICAL_CODE TRUE)
endif ()
Expand Down Expand Up @@ -102,7 +104,7 @@ if (ENABLE_BLAS3_FRNTEND_CNTL_TREES)
endif ()

if (ENABLE_WINDOWS_BUILD)
add_compile_definitions(FLA_ENABLE_WINDOWS_BUILD=1)
set(FLA_ENABLE_WINDOWS_BUILD TRUE)
endif ()

if (ENABLE_PORTABLE_TIMER)
Expand Down
4 changes: 2 additions & 2 deletions src/base/flamec/include/FLA_f2c.h
Original file line number Diff line number Diff line change
Expand Up @@ -1746,7 +1746,7 @@ typedef doublereal E_f; /* real function with -R not specified */
#define sorgrq_ SORGRQ
#define sorgtr_ SORGTR
#define sorgtsqr_ SORGTSQR
#define sorhr_col_ SORHR_OL
#define sorhr_col_ SORHR_COL
#define sorm22_ SORM22
#define sorm2l_ SORM2L
#define sorm2r_ SORM2R
Expand Down Expand Up @@ -3888,7 +3888,7 @@ typedef doublereal E_f; /* real function with -R not specified */
#define sorgrq_ sorgrq
#define sorgtr_ sorgtr
#define sorgtsqr_ sorgtsqr
#define sorhr_col_ sorhr_ol
#define sorhr_col_ sorhr_col
#define sorm22_ sorm22
#define sorm2l_ sorm2l
#define sorm2r_ sorm2r
Expand Down
3 changes: 3 additions & 0 deletions src/base/flamec/include/FLA_macro_defs.h
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,9 @@
#if !defined FLA_ENABLE_MULTITHREADING && !defined FLA_ENABLE_SUPERMATRIX
#define ENABLE_THREAD_LOCAL_STORAGE 1
#define LIBFLAME_IMPORT
#else
#define ENABLE_THREAD_LOCAL_STORAGE 0
#define LIBFLAME_IMPORT
#endif
#endif

Expand Down

0 comments on commit 593e464

Please sign in to comment.