[Mesa-dev] [PATCH 44/47] clover/core: Remove compiler.hpp.

Francisco Jerez currojerez at riseup.net
Mon Jul 4 00:51:53 UTC 2016


header_map was the only definition left in compiler.hpp, move it into
program.hpp which is its only user in clover/core.

Reviewed-by: Serge Martin <edb+mesa at sigluy.net>
---
 src/gallium/state_trackers/clover/Makefile.sources |  1 -
 .../state_trackers/clover/core/compiler.hpp        | 34 ----------------------
 src/gallium/state_trackers/clover/core/program.hpp |  3 +-
 .../state_trackers/clover/llvm/invocation.hpp      |  2 +-
 4 files changed, 3 insertions(+), 37 deletions(-)
 delete mode 100644 src/gallium/state_trackers/clover/core/compiler.hpp

diff --git a/src/gallium/state_trackers/clover/Makefile.sources b/src/gallium/state_trackers/clover/Makefile.sources
index a474130..e9828b1 100644
--- a/src/gallium/state_trackers/clover/Makefile.sources
+++ b/src/gallium/state_trackers/clover/Makefile.sources
@@ -13,7 +13,6 @@ CPP_SOURCES := \
 	api/sampler.cpp \
 	api/transfer.cpp \
 	api/util.hpp \
-	core/compiler.hpp \
 	core/context.cpp \
 	core/context.hpp \
 	core/device.cpp \
diff --git a/src/gallium/state_trackers/clover/core/compiler.hpp b/src/gallium/state_trackers/clover/core/compiler.hpp
deleted file mode 100644
index 572a9f4..0000000
--- a/src/gallium/state_trackers/clover/core/compiler.hpp
+++ /dev/null
@@ -1,34 +0,0 @@
-//
-// Copyright 2012 Francisco Jerez
-//
-// Permission is hereby granted, free of charge, to any person obtaining a
-// copy of this software and associated documentation files (the "Software"),
-// to deal in the Software without restriction, including without limitation
-// the rights to use, copy, modify, merge, publish, distribute, sublicense,
-// and/or sell copies of the Software, and to permit persons to whom the
-// Software is furnished to do so, subject to the following conditions:
-//
-// The above copyright notice and this permission notice shall be included in
-// all copies or substantial portions of the Software.
-//
-// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
-// THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
-// OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
-// ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR
-// OTHER DEALINGS IN THE SOFTWARE.
-//
-
-#ifndef CLOVER_CORE_COMPILER_HPP
-#define CLOVER_CORE_COMPILER_HPP
-
-#include "core/error.hpp"
-#include "core/module.hpp"
-#include "pipe/p_defines.h"
-
-namespace clover {
-   typedef std::vector<std::pair<std::string, std::string> > header_map;
-}
-
-#endif
diff --git a/src/gallium/state_trackers/clover/core/program.hpp b/src/gallium/state_trackers/clover/core/program.hpp
index 95dfd8e..76f16d2 100644
--- a/src/gallium/state_trackers/clover/core/program.hpp
+++ b/src/gallium/state_trackers/clover/core/program.hpp
@@ -28,9 +28,10 @@
 #include "core/object.hpp"
 #include "core/context.hpp"
 #include "core/module.hpp"
-#include "core/compiler.hpp"
 
 namespace clover {
+   typedef std::vector<std::pair<std::string, std::string>> header_map;
+
    class program : public ref_counter, public _cl_program {
    private:
       typedef adaptor_range<
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.hpp b/src/gallium/state_trackers/clover/llvm/invocation.hpp
index 9e90c50..5b3530c 100644
--- a/src/gallium/state_trackers/clover/llvm/invocation.hpp
+++ b/src/gallium/state_trackers/clover/llvm/invocation.hpp
@@ -23,9 +23,9 @@
 #ifndef CLOVER_LLVM_INVOCATION_HPP
 #define CLOVER_LLVM_INVOCATION_HPP
 
-#include "core/compiler.hpp"
 #include "core/error.hpp"
 #include "core/module.hpp"
+#include "core/program.hpp"
 #include "pipe/p_defines.h"
 
 namespace clover {
-- 
2.9.0



More information about the mesa-dev mailing list