[Mesa-dev] [PATCH] i965: Replace 'noann' debug flag with 'ann'.

Matt Turner mattst88 at gmail.com
Fri Dec 5 17:08:40 PST 2014


---
Eric was against making this the default when I first suggested a flag.
Have opinions changed since then? I rarely use the annotations, and they
do make the assembly harder to read, when the assembly is what you're
interested in.

 src/mesa/drivers/dri/i965/intel_asm_annotation.c | 2 +-
 src/mesa/drivers/dri/i965/intel_debug.c          | 2 +-
 src/mesa/drivers/dri/i965/intel_debug.h          | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/intel_asm_annotation.c b/src/mesa/drivers/dri/i965/intel_asm_annotation.c
index 37ad090..ac12655 100644
--- a/src/mesa/drivers/dri/i965/intel_asm_annotation.c
+++ b/src/mesa/drivers/dri/i965/intel_asm_annotation.c
@@ -109,7 +109,7 @@ void annotate(struct brw_context *brw,
 
    struct annotation *ann = &annotation->ann[annotation->ann_count++];
    ann->offset = offset;
-   if ((INTEL_DEBUG & DEBUG_NO_ANNOTATION) == 0) {
+   if ((INTEL_DEBUG & DEBUG_ANNOTATION) != 0) {
       ann->ir = inst->ir;
       ann->annotation = inst->annotation;
    }
diff --git a/src/mesa/drivers/dri/i965/intel_debug.c b/src/mesa/drivers/dri/i965/intel_debug.c
index 6391cf7..1dd2b1d 100644
--- a/src/mesa/drivers/dri/i965/intel_debug.c
+++ b/src/mesa/drivers/dri/i965/intel_debug.c
@@ -66,7 +66,7 @@ static const struct dri_debug_control debug_control[] = {
    { "blorp",       DEBUG_BLORP },
    { "nodualobj",   DEBUG_NO_DUAL_OBJECT_GS },
    { "optimizer",   DEBUG_OPTIMIZER },
-   { "noann",       DEBUG_NO_ANNOTATION },
+   { "ann",         DEBUG_ANNOTATION },
    { "no8",         DEBUG_NO8 },
    { NULL,    0 }
 };
diff --git a/src/mesa/drivers/dri/i965/intel_debug.h b/src/mesa/drivers/dri/i965/intel_debug.h
index e859be1..a01de21 100644
--- a/src/mesa/drivers/dri/i965/intel_debug.h
+++ b/src/mesa/drivers/dri/i965/intel_debug.h
@@ -61,7 +61,7 @@ extern uint64_t INTEL_DEBUG;
 #define DEBUG_VUE                 (1 << 25)
 #define DEBUG_NO_DUAL_OBJECT_GS   (1 << 26)
 #define DEBUG_OPTIMIZER           (1 << 27)
-#define DEBUG_NO_ANNOTATION       (1 << 28)
+#define DEBUG_ANNOTATION          (1 << 28)
 #define DEBUG_NO8                 (1 << 29)
 
 #ifdef HAVE_ANDROID_PLATFORM
-- 
2.0.4



More information about the mesa-dev mailing list