qf/quakeforge
0
0
Fork 0
mirror of https://git.code.sf.net/p/quake/quakeforge synced 2025-04-19 07:51:08 +00:00
Code Issues Projects Releases Packages Wiki Activity

[qfcc] Use statement aux def if def not in operands

Browse source 
This takes care of ptrmove instructions, fixing the printf problem in
ptrstructinit. It might even make it possible to not break basic blocks
on function calls, which should make for some more optimization
opportunities, but that can come later.

For this to work properly, it was necessary to avoid converting alias
defs to st_alias nodes.

It also fixes ptrstructinit itself (at first, I hadn't noticed that the
test passed entirely).
This commit is contained in:
Bill Currie 2024-02-19 23:13:49 +09:00
parent 07ef862c52
commit 2be9821a59
1 changed files with 11 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
tools/qfcc/source/dags.c
View file

@ -422,6 +422,14 @@ dag_make_children (dag_t *dag, statement_t *s,
int i;
flow_analyze_statement (s, 0, 0, 0, operands);
if (!operands[0] && s->def) {
auto op = s->def;
while (op && op->op_type == op_pseudo) {
op = op->next;
}
//FIXME hopefully only one non-pseudo op
operands[0] = op;
}
for (i = 0; i < 3; i++) {
operand_t *op = operands[i + 1];
dagnode_t *node = dag_node (op);
@ -436,7 +444,7 @@ dag_make_children (dag_t *dag, statement_t *s,
node = 0;
}
if (!node && op_is_alias (op)) {
if (!node && op_is_temp (op) && op_is_alias (op)) {
operand_t *uop = unalias_op (op);
if (uop != op) {
if (!(node = dag_node (uop))) {
@ -1005,6 +1013,7 @@ dag_create (flownode_t *flownode)
for (s = block->statements; s; s = s->next) {
num_statements++;
num_aux += dag_count_ops (s->use);
num_aux += dag_count_ops (s->def);
num_aux += op_is_alias (s->opa);
num_aux += op_is_alias (s->opb);
num_aux += op_is_alias (s->opc);
@ -1029,6 +1038,7 @@ dag_create (flownode_t *flownode)
dag_make_leafs (dag, s, operands);
// make sure any auxiliary operands are given nodes, too
dag_make_op_leafs (s->use, dag, s->expr);
dag_make_op_leafs (s->def, dag, s->expr);
}
// actual dag creation
for (s = block->statements; s; s = s->next) {