r600: Cleanup "nir" debug option.

Submitted by Bas Nieuwenhuizen on May 26, 2019, 7:45 p.m.

Details

Message ID 20190526194510.8055-1-bas@basnieuwenhuizen.nl
State New
Headers show
Series "r600: Cleanup "nir" debug option." ( rev: 1 ) in Mesa

Not browsing as part of any series.

Commit Message

Bas Nieuwenhuizen May 26, 2019, 7:45 p.m.
r600g does not have a nir compiler, and radeonsi does not use the
option either.
---
 src/gallium/drivers/r600/r600_pipe_common.c | 1 -
 src/gallium/drivers/r600/r600_pipe_common.h | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

Patch hide | download patch | download mbox

diff --git a/src/gallium/drivers/r600/r600_pipe_common.c b/src/gallium/drivers/r600/r600_pipe_common.c
index 5177ff4e1c6..b1ad0d7d23b 100644
--- a/src/gallium/drivers/r600/r600_pipe_common.c
+++ b/src/gallium/drivers/r600/r600_pipe_common.c
@@ -741,7 +741,6 @@  void r600_common_context_cleanup(struct r600_common_context *rctx)
 static const struct debug_named_value common_debug_options[] = {
 	/* logging */
 	{ "tex", DBG_TEX, "Print texture info" },
-	{ "nir", DBG_NIR, "Enable experimental NIR shaders" },
 	{ "compute", DBG_COMPUTE, "Print compute info" },
 	{ "vm", DBG_VM, "Print virtual addresses when creating resources" },
 	{ "info", DBG_INFO, "Print driver information" },
diff --git a/src/gallium/drivers/r600/r600_pipe_common.h b/src/gallium/drivers/r600/r600_pipe_common.h
index b43b7eecd10..4136ba5b565 100644
--- a/src/gallium/drivers/r600/r600_pipe_common.h
+++ b/src/gallium/drivers/r600/r600_pipe_common.h
@@ -78,7 +78,7 @@  struct u_log_context;
 #define DBG_ALL_SHADERS		(DBG_FS - 1)
 #define DBG_FS			(1 << 6) /* fetch shader */
 #define DBG_TEX			(1 << 7)
-#define DBG_NIR			(1 << 8)
+/* gap */
 #define DBG_COMPUTE		(1 << 9)
 /* gap */
 #define DBG_VM			(1 << 11)

Comments

On So, 2019-05-26 at 21:45 +0200, Bas Nieuwenhuizen wrote:
> r600g does not have a nir compiler, 
I'm working on this in an out of tree bruch, but well, I can always re-
introduce the flag when I start to upstream this, so keep it or leave
it. 

Ack-by: Gert Wollny <gert.wollny@collabora.com> 


> and radeonsi does not use the
> option either.
> ---
>  src/gallium/drivers/r600/r600_pipe_common.c | 1 -
>  src/gallium/drivers/r600/r600_pipe_common.h | 2 +-
>  2 files changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/src/gallium/drivers/r600/r600_pipe_common.c
> b/src/gallium/drivers/r600/r600_pipe_common.c
> index 5177ff4e1c6..b1ad0d7d23b 100644
> --- a/src/gallium/drivers/r600/r600_pipe_common.c
> +++ b/src/gallium/drivers/r600/r600_pipe_common.c
> @@ -741,7 +741,6 @@ void r600_common_context_cleanup(struct
> r600_common_context *rctx)
>  static const struct debug_named_value common_debug_options[] = {
>  	/* logging */
>  	{ "tex", DBG_TEX, "Print texture info" },
> -	{ "nir", DBG_NIR, "Enable experimental NIR shaders" },
>  	{ "compute", DBG_COMPUTE, "Print compute info" },
>  	{ "vm", DBG_VM, "Print virtual addresses when creating
> resources" },
>  	{ "info", DBG_INFO, "Print driver information" },
> diff --git a/src/gallium/drivers/r600/r600_pipe_common.h
> b/src/gallium/drivers/r600/r600_pipe_common.h
> index b43b7eecd10..4136ba5b565 100644
> --- a/src/gallium/drivers/r600/r600_pipe_common.h
> +++ b/src/gallium/drivers/r600/r600_pipe_common.h
> @@ -78,7 +78,7 @@ struct u_log_context;
>  #define DBG_ALL_SHADERS		(DBG_FS - 1)
>  #define DBG_FS			(1 << 6) /* fetch shader */
>  #define DBG_TEX			(1 << 7)
> -#define DBG_NIR			(1 << 8)
> +/* gap */
>  #define DBG_COMPUTE		(1 << 9)
>  /* gap */
>  #define DBG_VM			(1 << 11)