Skip to content

Commit

Permalink
Cleanup.
Browse files Browse the repository at this point in the history
  • Loading branch information
trivialfis committed Oct 10, 2024
1 parent 19f4191 commit 7772803
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 12 deletions.
1 change: 0 additions & 1 deletion src/common/timer.cc
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@

#include "../collective/communicator-inl.h"
#include "cuda_rt_utils.h"
#include <unistd.h> // for getpid

#if defined(XGBOOST_USE_NVTX)
#include <nvtx3/nvtx3.hpp>
Expand Down
2 changes: 0 additions & 2 deletions src/tree/updater_gpu_hist.cu
Original file line number Diff line number Diff line change
Expand Up @@ -738,11 +738,9 @@ struct GPUHistMakerDevice {
std::copy_if(expand_set.begin(), expand_set.end(), std::back_inserter(valid_candidates),
[&](auto const& e) { return driver.IsChildValid(e); });

monitor.Start("new_candidates");
// Allocaate children nodes.
auto new_candidates =
pinned.GetSpan<GPUExpandEntry>(valid_candidates.size() * 2, GPUExpandEntry());
monitor.Stop("new_candidates");

this->PartitionAndBuildHist(p_fmat, expand_set, valid_candidates, p_tree);

Expand Down
13 changes: 4 additions & 9 deletions tests/ci_build/build_via_cmake.sh
Original file line number Diff line number Diff line change
Expand Up @@ -21,17 +21,12 @@ else
cmake_prefix_flag=''
fi

# rm -rf build
# mkdir build

if [ ! -d build ]; then
mkdir build
fi

rm -rf build
mkdir build
cd build
# Disable CMAKE_COMPILE_WARNING_AS_ERROR option temporarily until
# https://github.com/dmlc/xgboost/issues/10400 is fixed
echo "cmake: ${cmake_args}"
cmake .. ${cmake_args} -DCMAKE_VERBOSE_MAKEFILE=ON -DENABLE_ALL_WARNINGS=ON -DCMAKE_COMPILE_WARNING_AS_ERROR=OFF -GNinja ${cmake_prefix_flag} -DHIDE_CXX_SYMBOLS=ON -DCMAKE_BUILD_TYPE=RelWithDebInfo
cmake .. ${cmake_args} -DGOOGLE_TEST=ON -DUSE_DMLC_GTEST=ON -DCMAKE_VERBOSE_MAKEFILE=ON -DENABLE_ALL_WARNINGS=ON -DCMAKE_COMPILE_WARNING_AS_ERROR=OFF -GNinja ${cmake_prefix_flag} -DHIDE_CXX_SYMBOLS=ON -DBUILD_DEPRECATED_CLI=ON
ninja clean
time ninja -v
cd ..

0 comments on commit 7772803

Please sign in to comment.