i965/blorp: Move exec() out of brw_blorp_params.
authorPaul Berry <stereotype441@gmail.com>
Tue, 15 May 2012 14:29:26 +0000 (07:29 -0700)
committerPaul Berry <stereotype441@gmail.com>
Tue, 15 May 2012 22:15:26 +0000 (15:15 -0700)
No functional change.  This patch replaces the
brw_blorp_params::exec() method with a global function
brw_blorp_exec() that performs the operation described by the params
data structure.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/brw_blorp.cpp
src/mesa/drivers/dri/i965/brw_blorp.h
src/mesa/drivers/dri/i965/brw_blorp_blit.cpp

index 8e225117fabea3e87c946925432b8b9e62143b68..994d646296c5f960e8851c22041557316eadb098 100644 (file)
@@ -99,14 +99,14 @@ brw_blorp_params::brw_blorp_params()
 }
 
 void
-brw_blorp_params::exec(struct intel_context *intel) const
+brw_blorp_exec(struct intel_context *intel, const brw_blorp_params *params)
 {
    switch (intel->gen) {
    case 6:
-      gen6_blorp_exec(intel, this);
+      gen6_blorp_exec(intel, params);
       break;
    case 7:
-      gen7_blorp_exec(intel, this);
+      gen7_blorp_exec(intel, params);
       break;
    default:
       /* BLORP is not supported before Gen6. */
index f14a5c7aae02c95f4d0de21446a65da8d81141dc..65662c068f89b1e6b63b169d1097d4bc3a7eb715 100644 (file)
@@ -142,8 +142,6 @@ public:
    virtual uint32_t get_wm_prog(struct brw_context *brw,
                                 brw_blorp_prog_data **prog_data) const = 0;
 
-   void exec(struct intel_context *intel) const;
-
    uint32_t x0;
    uint32_t y0;
    uint32_t x1;
@@ -158,6 +156,11 @@ public:
    brw_blorp_wm_push_constants wm_push_consts;
 };
 
+
+void
+brw_blorp_exec(struct intel_context *intel, const brw_blorp_params *params);
+
+
 /**
  * Parameters for a HiZ or depth resolve operation.
  *
index 1f0c3185394cc5f354b2e0f26206a46c262bd50e..ad31f4090e03a37761a84b255209885d7725edf5 100644 (file)
@@ -173,7 +173,7 @@ try_blorp_blit(struct intel_context *intel,
    brw_blorp_blit_params params(src_mt, dst_mt,
                                 srcX0, srcY0, dstX0, dstY0, dstX1, dstY1,
                                 mirror_x, mirror_y);
-   params.exec(intel);
+   brw_blorp_exec(intel, &params);
 
    /* Mark the dst buffer as needing a HiZ resolve if necessary. */
    intel_renderbuffer_set_needs_hiz_resolve(dst_irb);