vc4: Add an initial pass of algebraic optimization.
There was a lot of extra noise in my piglit shader dumps because of silly CMPs.
This commit is contained in:
@@ -4,6 +4,7 @@ C_SOURCES := \
|
||||
vc4_context.c \
|
||||
vc4_draw.c \
|
||||
vc4_emit.c \
|
||||
vc4_opt_algebraic.c \
|
||||
vc4_program.c \
|
||||
vc4_qir.c \
|
||||
vc4_qpu_emit.c \
|
||||
|
77
src/gallium/drivers/vc4/vc4_opt_algebraic.c
Normal file
77
src/gallium/drivers/vc4/vc4_opt_algebraic.c
Normal file
@@ -0,0 +1,77 @@
|
||||
/*
|
||||
* Copyright © 2014 Broadcom
|
||||
*
|
||||
* 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 (including the next
|
||||
* paragraph) 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.
|
||||
*/
|
||||
|
||||
/**
|
||||
* @file vc4_opt_algebraic.c
|
||||
*
|
||||
* This is the optimization pass for miscellaneous changes to instructions
|
||||
* where we can simplify the operation by some knowledge about the specific
|
||||
* operations.
|
||||
*
|
||||
* Mostly this will be a matter of turning things into MOVs so that they can
|
||||
* later be copy-propagated out.
|
||||
*/
|
||||
|
||||
#include <stdio.h>
|
||||
#include "vc4_qir.h"
|
||||
|
||||
bool
|
||||
qir_opt_algebraic(struct qcompile *c)
|
||||
{
|
||||
bool progress = false;
|
||||
struct simple_node *node;
|
||||
bool debug = false;
|
||||
|
||||
foreach(node, &c->instructions) {
|
||||
struct qinst *inst = (struct qinst *)node;
|
||||
|
||||
switch (inst->op) {
|
||||
case QOP_CMP:
|
||||
/* Turn "dst = (a < 0) ? b : b)" into "dst = b" */
|
||||
if (qir_reg_equals(inst->src[1], inst->src[2])) {
|
||||
if (debug) {
|
||||
fprintf(stderr, "optimizing: ");
|
||||
qir_dump_inst(inst);
|
||||
fprintf(stderr, "\n");
|
||||
}
|
||||
|
||||
inst->op = QOP_MOV;
|
||||
inst->src[0] = inst->src[1];
|
||||
inst->src[1] = c->undef;
|
||||
progress = true;
|
||||
|
||||
if (debug) {
|
||||
fprintf(stderr, "to: ");
|
||||
qir_dump_inst(inst);
|
||||
fprintf(stderr, "\n");
|
||||
}
|
||||
}
|
||||
break;
|
||||
|
||||
default:
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
return progress;
|
||||
}
|
@@ -593,14 +593,15 @@ vc4_shader_tgsi_to_qir(struct vc4_compiled_shader *shader, enum qstage stage,
|
||||
break;
|
||||
}
|
||||
|
||||
tgsi_parse_free(&trans->parser);
|
||||
free(trans->temps);
|
||||
|
||||
qir_optimize(c);
|
||||
|
||||
if (vc4_debug & VC4_DEBUG_QIR) {
|
||||
fprintf(stderr, "QIR:\n");
|
||||
qir_dump(c);
|
||||
}
|
||||
|
||||
tgsi_parse_free(&trans->parser);
|
||||
free(trans->temps);
|
||||
|
||||
vc4_generate_code(c);
|
||||
|
||||
if (vc4_debug & VC4_DEBUG_SHADERDB) {
|
||||
|
@@ -171,6 +171,12 @@ qir_emit(struct qcompile *c, struct qinst *inst)
|
||||
insert_at_tail(&c->instructions, &inst->link);
|
||||
}
|
||||
|
||||
bool
|
||||
qir_reg_equals(struct qreg a, struct qreg b)
|
||||
{
|
||||
return a.file == b.file && a.index == b.index;
|
||||
}
|
||||
|
||||
struct qcompile *
|
||||
qir_compile_init(void)
|
||||
{
|
||||
@@ -198,3 +204,34 @@ qir_get_stage_name(enum qstage stage)
|
||||
|
||||
return names[stage];
|
||||
}
|
||||
|
||||
#define OPTPASS(func) \
|
||||
do { \
|
||||
bool stage_progress = func(c); \
|
||||
if (stage_progress) { \
|
||||
progress = true; \
|
||||
if (print_opt_debug) { \
|
||||
fprintf(stderr, \
|
||||
"QIR opt pass %2d: %s progress\n", \
|
||||
pass, #func); \
|
||||
} \
|
||||
} \
|
||||
} while (0)
|
||||
|
||||
void
|
||||
qir_optimize(struct qcompile *c)
|
||||
{
|
||||
bool print_opt_debug = false;
|
||||
int pass = 1;
|
||||
|
||||
while (true) {
|
||||
bool progress = false;
|
||||
|
||||
OPTPASS(qir_opt_algebraic);
|
||||
|
||||
if (!progress)
|
||||
break;
|
||||
|
||||
pass++;
|
||||
}
|
||||
}
|
||||
|
@@ -24,6 +24,7 @@
|
||||
#ifndef VC4_QIR_H
|
||||
#define VC4_QIR_H
|
||||
|
||||
#include <stdbool.h>
|
||||
#include <stdint.h>
|
||||
|
||||
#include "util/u_simple_list.h"
|
||||
@@ -145,11 +146,15 @@ struct qinst *qir_inst4(enum qop op, struct qreg dst,
|
||||
void qir_emit(struct qcompile *c, struct qinst *inst);
|
||||
struct qreg qir_get_temp(struct qcompile *c);
|
||||
int qir_get_op_nsrc(enum qop qop);
|
||||
bool qir_reg_equals(struct qreg a, struct qreg b);
|
||||
|
||||
void qir_dump(struct qcompile *c);
|
||||
void qir_dump_inst(struct qinst *inst);
|
||||
const char *qir_get_stage_name(enum qstage stage);
|
||||
|
||||
void qir_optimize(struct qcompile *c);
|
||||
bool qir_opt_algebraic(struct qcompile *c);
|
||||
|
||||
#define QIR_ALU1(name) \
|
||||
static inline struct qreg \
|
||||
qir_##name(struct qcompile *c, struct qreg a) \
|
||||
|
Reference in New Issue
Block a user