OpenHarmony/third_party_mesa3d
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

intel/fs: Make opt_copy_propagation_local file private

Browse Source 
This annoyed me durning development of this MR. Every time I changed the
parameters to this internal function, I had to modify a public header
file... and trigger a much large rebuild.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25091>
This commit is contained in:
Ian Romanick
2023-08-18 17:31:12 -07:00
committed by Marge Bot
parent 0946108298
commit c262752d74
2 changed files with 7 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/intel/compiler/brw_fs.h
View File

@@ -263,8 +263,6 @@ public:
bool opt_cse_local(const brw::fs_live_variables &live, bblock_t *block, int &ip); bool opt_cse_local(const brw::fs_live_variables &live, bblock_t *block, int &ip);
bool opt_copy_propagation(); bool opt_copy_propagation();
bool opt_copy_propagation_local(void *mem_ctx, bblock_t *block,
exec_list *acp);
bool opt_register_renaming(); bool opt_register_renaming();
bool opt_bank_conflicts(); bool opt_bank_conflicts();
bool opt_split_sends(); bool opt_split_sends();

13
src/intel/compiler/brw_fs_copy_propagation.cpp
View File

@@ -1147,9 +1147,10 @@ can_propagate_from(fs_inst *inst)
/* Walks a basic block and does copy propagation on it using the acp /* Walks a basic block and does copy propagation on it using the acp
* list. * list.
*/ */
bool static bool
fs_visitor::opt_copy_propagation_local(void *copy_prop_ctx, bblock_t *block, opt_copy_propagation_local(const brw_compiler *compiler, void *copy_prop_ctx,
exec_list *acp) bblock_t *block, exec_list *acp,
const brw::simple_allocator &alloc)
{ {
bool progress = false; bool progress = false;
@@ -1285,8 +1286,8 @@ fs_visitor::opt_copy_propagation()
* the set of copies available at the end of the block. * the set of copies available at the end of the block.
*/ */
foreach_block (block, cfg) { foreach_block (block, cfg) {
progress = opt_copy_propagation_local(copy_prop_ctx, block, progress = opt_copy_propagation_local(compiler, copy_prop_ctx, block,
out_acp[block->num]) || progress; out_acp[block->num], alloc) || progress;
/* If the destination of an ACP entry exists only within this block, /* If the destination of an ACP entry exists only within this block,
* then there's no need to keep it for dataflow analysis. We can delete * then there's no need to keep it for dataflow analysis. We can delete
@@ -1325,7 +1326,7 @@ fs_visitor::opt_copy_propagation()
} }
} }
progress = opt_copy_propagation_local(copy_prop_ctx, block, in_acp) || progress = opt_copy_propagation_local(compiler, copy_prop_ctx, block, in_acp, alloc) ||
progress; progress;
} }