[Mesa-dev] [PATCH] Revert "i965: Combine assembly annotations if possible."
Kenneth Graunke
kenneth at whitecape.org
Tue Nov 24 02:55:11 PST 2015
This reverts commit a280e83d71bb046098ed5380cb053318f9e8cf8e.
It breaks INTEL_DEBUG=fs output. For example,
glsl-fs-discard-01.shader_test has 11 instructions but only prints 5.
---
src/mesa/drivers/dri/i965/intel_asm_annotation.c | 23 +++++------------------
1 file changed, 5 insertions(+), 18 deletions(-)
We should probably revert this for now, and re-apply a v2...
diff --git a/src/mesa/drivers/dri/i965/intel_asm_annotation.c b/src/mesa/drivers/dri/i965/intel_asm_annotation.c
index fdd605a..59cc863 100644
--- a/src/mesa/drivers/dri/i965/intel_asm_annotation.c
+++ b/src/mesa/drivers/dri/i965/intel_asm_annotation.c
@@ -121,24 +121,6 @@ void annotate(const struct brw_device_info *devinfo,
ann->block_start = cfg->blocks[annotation->cur_block];
}
- if (bblock_end(cfg->blocks[annotation->cur_block]) == inst) {
- ann->block_end = cfg->blocks[annotation->cur_block];
- annotation->cur_block++;
- }
-
- /* Merge this annotation with the previous if possible. */
- struct annotation *prev = annotation->ann_count > 1 ?
- &annotation->ann[annotation->ann_count - 2] : NULL;
- if (prev != NULL &&
- ann->ir == prev->ir &&
- ann->annotation == prev->annotation &&
- ann->block_start == NULL &&
- prev->block_end == NULL) {
- if (ann->block_end == NULL)
- annotation->ann_count--;
- return;
- }
-
/* There is no hardware DO instruction on Gen6+, so since DO always
* starts a basic block, we need to set the .block_start of the next
* instruction's annotation with a pointer to the bblock started by
@@ -150,6 +132,11 @@ void annotate(const struct brw_device_info *devinfo,
if (devinfo->gen >= 6 && inst->opcode == BRW_OPCODE_DO) {
annotation->ann_count--;
}
+
+ if (bblock_end(cfg->blocks[annotation->cur_block]) == inst) {
+ ann->block_end = cfg->blocks[annotation->cur_block];
+ annotation->cur_block++;
+ }
}
void
--
2.6.2
More information about the mesa-dev
mailing list