Mesa (master): clover: Unabbreviate a few data accessor names for consistency.

Francisco Jerez currojerez at kemper.freedesktop.org
Fri Feb 21 11:58:20 UTC 2014


Module: Mesa
Branch: master
Commit: 9b2fe7cf96dc25f52ae43c8dfda20ee354c329f2
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=9b2fe7cf96dc25f52ae43c8dfda20ee354c329f2

Author: Francisco Jerez <currojerez at riseup.net>
Date:   Tue Feb 18 13:12:52 2014 +0100

clover: Unabbreviate a few data accessor names for consistency.

Tested-by: Tom Stellard <thomas.stellard at amd.com>

---

 src/gallium/state_trackers/clover/api/context.cpp  |    6 +++---
 src/gallium/state_trackers/clover/api/memory.cpp   |    2 +-
 src/gallium/state_trackers/clover/api/program.cpp  |   12 ++++++------
 src/gallium/state_trackers/clover/api/queue.cpp    |    4 ++--
 src/gallium/state_trackers/clover/core/context.cpp |   10 +++++-----
 src/gallium/state_trackers/clover/core/context.hpp |    8 ++++----
 src/gallium/state_trackers/clover/core/format.cpp  |    2 +-
 src/gallium/state_trackers/clover/core/queue.cpp   |    8 ++++----
 src/gallium/state_trackers/clover/core/queue.hpp   |    4 ++--
 9 files changed, 28 insertions(+), 28 deletions(-)

diff --git a/src/gallium/state_trackers/clover/api/context.cpp b/src/gallium/state_trackers/clover/api/context.cpp
index a78c00b..021eea3 100644
--- a/src/gallium/state_trackers/clover/api/context.cpp
+++ b/src/gallium/state_trackers/clover/api/context.cpp
@@ -117,15 +117,15 @@ clGetContextInfo(cl_context d_ctx, cl_context_info param,
       break;
 
    case CL_CONTEXT_NUM_DEVICES:
-      buf.as_scalar<cl_uint>() = ctx.devs().size();
+      buf.as_scalar<cl_uint>() = ctx.devices().size();
       break;
 
    case CL_CONTEXT_DEVICES:
-      buf.as_vector<cl_device_id>() = descs(ctx.devs());
+      buf.as_vector<cl_device_id>() = descs(ctx.devices());
       break;
 
    case CL_CONTEXT_PROPERTIES:
-      buf.as_vector<cl_context_properties>() = desc(ctx.props());
+      buf.as_vector<cl_context_properties>() = desc(ctx.properties());
       break;
 
    default:
diff --git a/src/gallium/state_trackers/clover/api/memory.cpp b/src/gallium/state_trackers/clover/api/memory.cpp
index 84b3bcb..7ed2191 100644
--- a/src/gallium/state_trackers/clover/api/memory.cpp
+++ b/src/gallium/state_trackers/clover/api/memory.cpp
@@ -38,7 +38,7 @@ clCreateBuffer(cl_context d_ctx, cl_mem_flags flags, size_t size,
 
    if (!size ||
        size > fold(maximum(), cl_ulong(0),
-                   map(std::mem_fn(&device::max_mem_alloc_size), ctx.devs())
+                   map(std::mem_fn(&device::max_mem_alloc_size), ctx.devices())
           ))
       throw error(CL_INVALID_BUFFER_SIZE);
 
diff --git a/src/gallium/state_trackers/clover/api/program.cpp b/src/gallium/state_trackers/clover/api/program.cpp
index 8867ee9..fedc91d 100644
--- a/src/gallium/state_trackers/clover/api/program.cpp
+++ b/src/gallium/state_trackers/clover/api/program.cpp
@@ -64,7 +64,7 @@ clCreateProgramWithBinary(cl_context d_ctx, cl_uint n,
       throw error(CL_INVALID_VALUE);
 
    if (any_of([&](const device &dev) {
-            return !count(dev, ctx.devs());
+            return !count(dev, ctx.devices());
          }, devs))
       throw error(CL_INVALID_DEVICE);
 
@@ -133,7 +133,7 @@ clBuildProgram(cl_program d_prog, cl_uint num_devs,
                void *user_data) try {
    auto &prog = obj(d_prog);
    auto devs = (d_devs ? objs(d_devs, num_devs) :
-                ref_vector<device>(prog.context().devs()));
+                ref_vector<device>(prog.context().devices()));
    auto opts = (p_opts ? p_opts : "");
 
    if (bool(num_devs) != bool(d_devs) ||
@@ -141,7 +141,7 @@ clBuildProgram(cl_program d_prog, cl_uint num_devs,
       throw error(CL_INVALID_VALUE);
 
    if (any_of([&](const device &dev) {
-            return !count(dev, prog.context().devs());
+            return !count(dev, prog.context().devices());
          }, devs))
       throw error(CL_INVALID_DEVICE);
 
@@ -175,13 +175,13 @@ clGetProgramInfo(cl_program d_prog, cl_program_info param,
    case CL_PROGRAM_NUM_DEVICES:
       buf.as_scalar<cl_uint>() = (prog.devices().size() ?
                                   prog.devices().size() :
-                                  prog.context().devs().size());
+                                  prog.context().devices().size());
       break;
 
    case CL_PROGRAM_DEVICES:
       buf.as_vector<cl_device_id>() = (prog.devices().size() ?
                                        descs(prog.devices()) :
-                                       descs(prog.context().devs()));
+                                       descs(prog.context().devices()));
       break;
 
    case CL_PROGRAM_SOURCE:
@@ -226,7 +226,7 @@ clGetProgramBuildInfo(cl_program d_prog, cl_device_id d_dev,
    auto &prog = obj(d_prog);
    auto &dev = obj(d_dev);
 
-   if (!count(dev, prog.context().devs()))
+   if (!count(dev, prog.context().devices()))
       return CL_INVALID_DEVICE;
 
    switch (param) {
diff --git a/src/gallium/state_trackers/clover/api/queue.cpp b/src/gallium/state_trackers/clover/api/queue.cpp
index 8498e97..a136018 100644
--- a/src/gallium/state_trackers/clover/api/queue.cpp
+++ b/src/gallium/state_trackers/clover/api/queue.cpp
@@ -32,7 +32,7 @@ clCreateCommandQueue(cl_context d_ctx, cl_device_id d_dev,
    auto &ctx = obj(d_ctx);
    auto &dev = obj(d_dev);
 
-   if (!count(dev, ctx.devs()))
+   if (!count(dev, ctx.devices()))
       throw error(CL_INVALID_DEVICE);
 
    if (props & ~(CL_QUEUE_OUT_OF_ORDER_EXEC_MODE_ENABLE |
@@ -87,7 +87,7 @@ clGetCommandQueueInfo(cl_command_queue d_q, cl_command_queue_info param,
       break;
 
    case CL_QUEUE_PROPERTIES:
-      buf.as_scalar<cl_command_queue_properties>() = q.props();
+      buf.as_scalar<cl_command_queue_properties>() = q.properties();
       break;
 
    default:
diff --git a/src/gallium/state_trackers/clover/core/context.cpp b/src/gallium/state_trackers/clover/core/context.cpp
index 34ffd9f..bf4df39 100644
--- a/src/gallium/state_trackers/clover/core/context.cpp
+++ b/src/gallium/state_trackers/clover/core/context.cpp
@@ -26,7 +26,7 @@ using namespace clover;
 
 context::context(const property_list &props,
                  const ref_vector<device> &devs) :
-   _props(props), _devs(devs) {
+   props(props), devs(devs) {
 }
 
 bool
@@ -40,11 +40,11 @@ context::operator!=(const context &ctx) const {
 }
 
 const context::property_list &
-context::props() const {
-   return _props;
+context::properties() const {
+   return props;
 }
 
 context::device_range
-context::devs() const {
-   return map(evals(), _devs);
+context::devices() const {
+   return map(evals(), devs);
 }
diff --git a/src/gallium/state_trackers/clover/core/context.hpp b/src/gallium/state_trackers/clover/core/context.hpp
index ef14f15..0ec4ff4 100644
--- a/src/gallium/state_trackers/clover/core/context.hpp
+++ b/src/gallium/state_trackers/clover/core/context.hpp
@@ -48,14 +48,14 @@ namespace clover {
       operator!=(const context &ctx) const;
 
       const property_list &
-      props() const;
+      properties() const;
 
       device_range
-      devs() const;
+      devices() const;
 
    private:
-      property_list _props;
-      const std::vector<intrusive_ref<device>> _devs;
+      property_list props;
+      const std::vector<intrusive_ref<device>> devs;
    };
 }
 
diff --git a/src/gallium/state_trackers/clover/core/format.cpp b/src/gallium/state_trackers/clover/core/format.cpp
index 7958230..aee1cc9 100644
--- a/src/gallium/state_trackers/clover/core/format.cpp
+++ b/src/gallium/state_trackers/clover/core/format.cpp
@@ -155,7 +155,7 @@ namespace clover {
          if (all_of([=](const device &dev) {
                   return dev.pipe->is_format_supported(
                      dev.pipe, f.second, target, 1, bindings);
-               }, ctx.devs()))
+               }, ctx.devices()))
             s.insert(f.first);
       }
 
diff --git a/src/gallium/state_trackers/clover/core/queue.cpp b/src/gallium/state_trackers/clover/core/queue.cpp
index 246d927..24f9326 100644
--- a/src/gallium/state_trackers/clover/core/queue.cpp
+++ b/src/gallium/state_trackers/clover/core/queue.cpp
@@ -29,7 +29,7 @@ using namespace clover;
 
 command_queue::command_queue(clover::context &ctx, clover::device &dev,
                              cl_command_queue_properties props) :
-   context(ctx), device(dev), _props(props) {
+   context(ctx), device(dev), props(props) {
    pipe = dev.pipe->context_create(dev.pipe, NULL);
    if (!pipe)
       throw error(CL_INVALID_DEVICE);
@@ -58,13 +58,13 @@ command_queue::flush() {
 }
 
 cl_command_queue_properties
-command_queue::props() const {
-   return _props;
+command_queue::properties() const {
+   return props;
 }
 
 bool
 command_queue::profiling_enabled() const {
-   return _props & CL_QUEUE_PROFILING_ENABLE;
+   return props & CL_QUEUE_PROFILING_ENABLE;
 }
 
 void
diff --git a/src/gallium/state_trackers/clover/core/queue.hpp b/src/gallium/state_trackers/clover/core/queue.hpp
index 19cb891..b7166e6 100644
--- a/src/gallium/state_trackers/clover/core/queue.hpp
+++ b/src/gallium/state_trackers/clover/core/queue.hpp
@@ -47,7 +47,7 @@ namespace clover {
 
       void flush();
 
-      cl_command_queue_properties props() const;
+      cl_command_queue_properties properties() const;
       bool profiling_enabled() const;
 
       const intrusive_ref<clover::context> context;
@@ -67,7 +67,7 @@ namespace clover {
       /// and push it to the pending list.
       void sequence(hard_event &ev);
 
-      cl_command_queue_properties _props;
+      cl_command_queue_properties props;
       pipe_context *pipe;
       std::deque<intrusive_ref<hard_event>> queued_events;
    };




More information about the mesa-commit mailing list