From 5edfc1e2e9952f5c92fd9e410d7b422b377804b8 Mon Sep 17 00:00:00 2001
From: amdsc21 <96135754+amdsc21@users.noreply.github.com>
Date: Fri, 10 Mar 2023 06:41:25 +0100
Subject: [PATCH] finish ellpack_page.cc

---
 src/data/ellpack_page.cc | 35 +----------------------------------
 1 file changed, 1 insertion(+), 34 deletions(-)

diff --git a/src/data/ellpack_page.cc b/src/data/ellpack_page.cc
index e3df86945..6199c1b21 100644
--- a/src/data/ellpack_page.cc
+++ b/src/data/ellpack_page.cc
@@ -1,7 +1,7 @@
 /*!
  * Copyright 2019 XGBoost contributors
  */
-#if !defined(XGBOOST_USE_CUDA)
+#if !defined(XGBOOST_USE_CUDA) && !defined(XGBOOST_USE_HIP)
 
 #include <xgboost/data.h>
 
@@ -32,38 +32,5 @@ size_t EllpackPage::Size() const {
   return 0;
 }
 
-}  // namespace xgboost
-
-#elif !defined(XGBOOST_USE_HIP)
-
-#include <xgboost/data.h>
-
-// dummy implementation of EllpackPage in case HIP is not used
-namespace xgboost {
-
-class EllpackPageImpl {};
-
-EllpackPage::EllpackPage() = default;
-
-EllpackPage::EllpackPage(DMatrix*, const BatchParam&) {
-  LOG(FATAL) << "Internal Error: XGBoost is not compiled with HIP but "
-                "EllpackPage is required";
-}
-
-EllpackPage::~EllpackPage() {
-  LOG(FATAL) << "Internal Error: XGBoost is not compiled with HIP but "
-                "EllpackPage is required";
-}
-
-void EllpackPage::SetBaseRowId(std::size_t) {
-  LOG(FATAL) << "Internal Error: XGBoost is not compiled with HIP but "
-                "EllpackPage is required";
-}
-size_t EllpackPage::Size() const {
-  LOG(FATAL) << "Internal Error: XGBoost is not compiled with HIP but "
-                "EllpackPage is required";
-  return 0;
-}
-
 }  // namespace xgboost
 #endif  // XGBOOST_USE_CUDA || XGBOOST_USE_HIP