Skip to content

Commit 03127d2

Browse files
committed
Build with CXX11 ABI only
Signed-off-by: Gigon Bae <[email protected]>
1 parent 4ddd932 commit 03127d2

File tree

6 files changed

+0
-36
lines changed

6 files changed

+0
-36
lines changed

CMakeLists.txt

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -96,10 +96,6 @@ option(CUCIM_STATIC_GDS "Use static cufile library" OFF)
9696
option(CUCIM_SUPPORT_CUDA "Support CUDA" ON)
9797
option(CUCIM_SUPPORT_NVTX "Support NVTX" ON)
9898

99-
# Setup CXX11 ABI
100-
# : Adds CXX11 ABI definition to the compiler command line for targets in the current directory,
101-
# whether added before or after this command is invoked, and for the ones in sub-directories added after.
102-
add_definitions(-D_GLIBCXX_USE_CXX11_ABI=0) # TODO: create two library, one with CXX11 ABI and one without it.
10399

104100
################################################################################
105101
# Define basic dependencies

cpp/CMakeLists.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,6 @@ target_compile_definitions(${CUCIM_PACKAGE_NAME}
139139
CUCIM_STATIC_GDS=$<BOOL:${CUCIM_STATIC_GDS}>
140140
CUCIM_SUPPORT_CUDA=$<BOOL:${CUCIM_SUPPORT_CUDA}>
141141
CUCIM_SUPPORT_NVTX=$<BOOL:${CUCIM_SUPPORT_NVTX}>
142-
_GLIBCXX_USE_CXX11_ABI=0 # TODO: create two library, one with CXX11 ABI and one without it.
143142
)
144143

145144
# Link libraries

cpp/include/cucim/io/format/image_format.h

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -119,10 +119,6 @@ class EXPORT_VISIBLE ImageMetadata
119119
std::array<uint8_t, IMAGE_METADATA_BUFFER_SIZE> buffer_{};
120120
std::pmr::monotonic_buffer_resource res_{ buffer_.data(), sizeof(buffer_) };
121121

122-
// manylinux2014 requires gcc4-compatible libstdcxx-abi(gcc is configured with
123-
// '--with-default-libstdcxx-abi=gcc4-compatible', https://gcc.gnu.org/onlinedocs/libstdc++/manual/configure.html) which
124-
// forces to set _GLIBCXX_USE_CXX11_ABI=0 so std::pmr::string wouldn't be available on CentOS 7.
125-
#if _GLIBCXX_USE_CXX11_ABI
126122
std::pmr::string dims_{ &res_ };
127123
std::pmr::vector<int64_t> shape_{ &res_ };
128124
std::pmr::vector<std::string_view> channel_names_{ &res_ };
@@ -137,22 +133,7 @@ class EXPORT_VISIBLE ImageMetadata
137133
std::pmr::vector<uint32_t> level_tile_sizes_{ &res_ };
138134

139135
std::pmr::vector<std::string_view> image_names_{ &res_ };
140-
#else
141-
std::string dims_;
142-
std::pmr::vector<int64_t> shape_{ &res_ };
143-
std::pmr::vector<std::string_view> channel_names_{ &res_ };
144-
std::pmr::vector<float> spacing_{ &res_ };
145-
std::pmr::vector<std::string_view> spacing_units_{ &res_ };
146-
std::pmr::vector<float> origin_{ &res_ };
147-
std::pmr::vector<float> direction_{ &res_ };
148-
std::string coord_sys_;
149136

150-
std::pmr::vector<int64_t> level_dimensions_{ &res_ };
151-
std::pmr::vector<float> level_downsamples_{ &res_ };
152-
std::pmr::vector<uint32_t> level_tile_sizes_{ &res_ };
153-
154-
std::pmr::vector<std::string_view> image_names_{ &res_ };
155-
#endif
156137
// Memory for raw_data and json_data needs to be created with cucim_malloc();
157138
};
158139

cpp/plugins/cucim.kit.cumed/CMakeLists.txt

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -105,10 +105,6 @@ include(ExternalProject)
105105
# Options
106106
################################################################################
107107

108-
# Setup CXX11 ABI
109-
# : Adds CXX11 ABI definition to the compiler command line for targets in the current directory,
110-
# whether added before or after this command is invoked, and for the ones in sub-directories added after.
111-
add_definitions(-D_GLIBCXX_USE_CXX11_ABI=0) # TODO: create two library, one with CXX11 ABI and one without it.
112108

113109
################################################################################
114110
# Define dependencies

cpp/plugins/cucim.kit.cuslide/CMakeLists.txt

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -105,10 +105,6 @@ include(ExternalProject)
105105
# Options
106106
################################################################################
107107

108-
# Setup CXX11 ABI
109-
# : Adds CXX11 ABI definition to the compiler command line for targets in the current directory,
110-
# whether added before or after this command is invoked, and for the ones in sub-directories added after.
111-
add_definitions(-D_GLIBCXX_USE_CXX11_ABI=0) # TODO: create two library, one with CXX11 ABI and one without it.
112108

113109
################################################################################
114110
# Define dependencies

python/CMakeLists.txt

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -89,10 +89,6 @@ include(ExternalProject)
8989
# Options
9090
################################################################################
9191

92-
# Setup CXX11 ABI
93-
# : Adds CXX11 ABI definition to the compiler command line for targets in the current directory,
94-
# whether added before or after this command is invoked, and for the ones in sub-directories added after.
95-
add_definitions(-D_GLIBCXX_USE_CXX11_ABI=0) # TODO: create two library, one with CXX11 ABI and one without it.
9692

9793
################################################################################
9894
# Define dependencies

0 commit comments

Comments
 (0)