freedreno/rnn: warnings cleanup

Signed-off-by: Rob Clark <robdclark@chromium.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6070>
This commit is contained in:
Rob Clark
2020-07-24 14:12:04 -07:00
committed by Marge Bot
parent 536f43cb96
commit cbbaafdf72
4 changed files with 14 additions and 9 deletions

View File

@@ -28,7 +28,6 @@
/* modified version of headergen which uses enums and inline fxns for /* modified version of headergen which uses enums and inline fxns for
* type safety.. based on original headergen * type safety.. based on original headergen
*/ */
#define _GNU_SOURCE
#include "rnn.h" #include "rnn.h"
#include "util.h" #include "util.h"
@@ -159,6 +158,8 @@ static void printtypeinfo (struct rnntypeinfo *ti, struct rnnbitfield *bf,
case RNN_TTYPE_ENUM: case RNN_TTYPE_ENUM:
asprintf(&typename, "enum %s", ti->name); asprintf(&typename, "enum %s", ti->name);
break; break;
default:
break;
} }
} }

View File

@@ -26,6 +26,9 @@
* OTHER DEALINGS IN THE SOFTWARE. * OTHER DEALINGS IN THE SOFTWARE.
*/ */
/* workaround libxml2 silliness: */
#pragma GCC diagnostic ignored "-Wpointer-sign"
#include <libxml/xmlversion.h> #include <libxml/xmlversion.h>
#include <libxml/parser.h> #include <libxml/parser.h>
#include <libxml/xpath.h> #include <libxml/xpath.h>

View File

@@ -23,7 +23,6 @@
* OTHER DEALINGS IN THE SOFTWARE. * OTHER DEALINGS IN THE SOFTWARE.
*/ */
#define _GNU_SOURCE // for asprintf
#include "rnndec.h" #include "rnndec.h"
#include <assert.h> #include <assert.h>
#include <stdio.h> #include <stdio.h>
@@ -39,7 +38,7 @@ struct rnndeccontext *rnndec_newcontext(struct rnndb *db) {
return res; return res;
} }
int rnndec_varadd(struct rnndeccontext *ctx, char *varset, char *variant) { int rnndec_varadd(struct rnndeccontext *ctx, char *varset, const char *variant) {
struct rnnenum *en = rnn_findenum(ctx->db, varset); struct rnnenum *en = rnn_findenum(ctx->db, varset);
if (!en) { if (!en) {
fprintf (stderr, "Enum %s doesn't exist in database!\n", varset); fprintf (stderr, "Enum %s doesn't exist in database!\n", varset);
@@ -168,6 +167,7 @@ char *rnndec_decodeval(struct rnndeccontext *ctx, struct rnntypeinfo *ti, uint64
struct rnnbitfield **bitfields; struct rnnbitfield **bitfields;
int bitfieldsnum; int bitfieldsnum;
char *tmp; char *tmp;
const char *ctmp;
uint64_t mask, value_orig; uint64_t mask, value_orig;
if (!ti) if (!ti)
goto failhex; goto failhex;
@@ -185,11 +185,11 @@ char *rnndec_decodeval(struct rnndeccontext *ctx, struct rnntypeinfo *ti, uint64
valsnum = ti->valsnum; valsnum = ti->valsnum;
goto doenum; goto doenum;
doenum: doenum:
tmp = rnndec_decode_enum_val(ctx, vals, valsnum, value); ctmp = rnndec_decode_enum_val(ctx, vals, valsnum, value);
if (tmp) { if (ctmp) {
asprintf (&res, "%s%s%s", ctx->colors->eval, tmp, ctx->colors->reset); asprintf (&res, "%s%s%s", ctx->colors->eval, ctmp, ctx->colors->reset);
if (ti->addvariant) { if (ti->addvariant) {
rnndec_varadd(ctx, ti->eenum->name, tmp); rnndec_varadd(ctx, ti->eenum->name, ctmp);
} }
break; break;
} }
@@ -325,7 +325,8 @@ char *rnndec_decodeval(struct rnndeccontext *ctx, struct rnntypeinfo *ti, uint64
} }
static char *appendidx (struct rnndeccontext *ctx, char *name, uint64_t idx, struct rnnenum *index) { static char *appendidx (struct rnndeccontext *ctx, char *name, uint64_t idx, struct rnnenum *index) {
char *res, *index_name = NULL; char *res;
const char *index_name = NULL;
if (index) if (index)
index_name = rnndec_decode_enum_val(ctx, index->vals, index->valsnum, idx); index_name = rnndec_decode_enum_val(ctx, index->vals, index->valsnum, idx);

View File

@@ -48,7 +48,7 @@ struct rnndecaddrinfo {
}; };
struct rnndeccontext *rnndec_newcontext(struct rnndb *db); struct rnndeccontext *rnndec_newcontext(struct rnndb *db);
int rnndec_varadd(struct rnndeccontext *ctx, char *varset, char *variant); int rnndec_varadd(struct rnndeccontext *ctx, char *varset, const char *variant);
int rnndec_varmatch(struct rnndeccontext *ctx, struct rnnvarinfo *vi); int rnndec_varmatch(struct rnndeccontext *ctx, struct rnnvarinfo *vi);
char *rnndec_decode_enum(struct rnndeccontext *ctx, const char *enumname, uint64_t enumval); char *rnndec_decode_enum(struct rnndeccontext *ctx, const char *enumname, uint64_t enumval);
char *rnndec_decodeval(struct rnndeccontext *ctx, struct rnntypeinfo *ti, uint64_t value); char *rnndec_decodeval(struct rnndeccontext *ctx, struct rnntypeinfo *ti, uint64_t value);