Skip to content

Commit

Permalink
Revert "Build dmlc-core with old thread_local implementation (apache#…
Browse files Browse the repository at this point in the history
…16526)"

This reverts commit 1fb6f00.
  • Loading branch information
larroy committed Oct 25, 2019
1 parent 67bb479 commit 94e72dd
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 86 deletions.
3 changes: 0 additions & 3 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -101,9 +101,6 @@ if("$ENV{VERBOSE}" STREQUAL "1")
set(CMAKE_VERBOSE_MAKEFILE ON)
endif()

#Switch off modern thread local for dmlc-core, please see: https://github.com/dmlc/dmlc-core/issues/571#issuecomment-543467484
add_definitions(-DDMLC_MODERN_THREAD_LOCAL=0)


if(MSVC)
add_definitions(-DWIN32_LEAN_AND_MEAN)
Expand Down
2 changes: 0 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -94,8 +94,6 @@ include $(DMLC_CORE)/make/dmlc.mk
# all tge possible warning tread
WARNFLAGS= -Wall -Wsign-compare
CFLAGS = -DMSHADOW_FORCE_STREAM $(WARNFLAGS)
# use old thread local implementation in DMLC-CORE
CFLAGS += -DDMLC_MODERN_THREAD_LOCAL=0

ifeq ($(DEV), 1)
CFLAGS += -g -Werror
Expand Down
80 changes: 0 additions & 80 deletions tests/cpp/engine/thread_local_test.cc

This file was deleted.

0 comments on commit 94e72dd

Please sign in to comment.