i965/fs: add a pass for lowering PACK opcodes
v2: Use subscript() instead of stride() (Curro) Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
This commit is contained in:
parent
ba582e58cd
commit
fd763177c1
|
@ -26,6 +26,7 @@ i965_compiler_FILES = \
|
||||||
brw_fs.h \
|
brw_fs.h \
|
||||||
brw_fs_live_variables.cpp \
|
brw_fs_live_variables.cpp \
|
||||||
brw_fs_live_variables.h \
|
brw_fs_live_variables.h \
|
||||||
|
brw_fs_lower_pack.cpp \
|
||||||
brw_fs_nir.cpp \
|
brw_fs_nir.cpp \
|
||||||
brw_fs_reg_allocate.cpp \
|
brw_fs_reg_allocate.cpp \
|
||||||
brw_fs_register_coalesce.cpp \
|
brw_fs_register_coalesce.cpp \
|
||||||
|
|
|
@ -5349,6 +5349,11 @@ fs_visitor::optimize()
|
||||||
OPT(dead_code_eliminate);
|
OPT(dead_code_eliminate);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if (OPT(lower_pack)) {
|
||||||
|
OPT(register_coalesce);
|
||||||
|
OPT(dead_code_eliminate);
|
||||||
|
}
|
||||||
|
|
||||||
OPT(opt_combine_constants);
|
OPT(opt_combine_constants);
|
||||||
OPT(lower_integer_multiplication);
|
OPT(lower_integer_multiplication);
|
||||||
|
|
||||||
|
|
|
@ -175,6 +175,7 @@ public:
|
||||||
void no16(const char *msg);
|
void no16(const char *msg);
|
||||||
void lower_uniform_pull_constant_loads();
|
void lower_uniform_pull_constant_loads();
|
||||||
bool lower_load_payload();
|
bool lower_load_payload();
|
||||||
|
bool lower_pack();
|
||||||
bool lower_logical_sends();
|
bool lower_logical_sends();
|
||||||
bool lower_integer_multiplication();
|
bool lower_integer_multiplication();
|
||||||
bool lower_minmax();
|
bool lower_minmax();
|
||||||
|
|
|
@ -0,0 +1,55 @@
|
||||||
|
/*
|
||||||
|
* Copyright © 2015 Connor Abbott
|
||||||
|
*
|
||||||
|
* 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.
|
||||||
|
*/
|
||||||
|
|
||||||
|
#include "brw_fs.h"
|
||||||
|
#include "brw_cfg.h"
|
||||||
|
#include "brw_fs_builder.h"
|
||||||
|
|
||||||
|
using namespace brw;
|
||||||
|
|
||||||
|
bool
|
||||||
|
fs_visitor::lower_pack()
|
||||||
|
{
|
||||||
|
bool progress = false;
|
||||||
|
|
||||||
|
foreach_block_and_inst_safe(block, fs_inst, inst, cfg) {
|
||||||
|
if (inst->opcode != FS_OPCODE_PACK)
|
||||||
|
continue;
|
||||||
|
|
||||||
|
assert(inst->dst.file == VGRF);
|
||||||
|
assert(inst->saturate == false);
|
||||||
|
fs_reg dst = inst->dst;
|
||||||
|
|
||||||
|
const fs_builder ibld(this, block, inst);
|
||||||
|
for (unsigned i = 0; i < inst->sources; i++)
|
||||||
|
ibld.MOV(subscript(dst, inst->src[i].type, i), inst->src[i]);
|
||||||
|
|
||||||
|
inst->remove(block);
|
||||||
|
progress = true;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (progress)
|
||||||
|
invalidate_live_intervals();
|
||||||
|
|
||||||
|
return progress;
|
||||||
|
}
|
Loading…
Reference in New Issue