From: Sirisha Gandikota <Sirisha.Gandikota@intel.com>
Date: Tue, 13 Sep 2016 23:19:29 +0000 (-0700)
Subject: aubinator: Simplify print_dword_val() method
X-Git-Tag: upstream/17.1.0~6416
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5d2440532f8ffe80faa9b818fa0c1abff312486f;p=platform%2Fupstream%2Fmesa.git

aubinator: Simplify print_dword_val() method

Remove the float/dword union and use the iter->p[f->start / 32]
directly as printf formatter %08x expects uint32_t (Ken)

v2: Make the cleanup much more crispier (Ken)

Signed-off-by: Sirisha Gandikota <Sirisha.Gandikota@intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
---

diff --git a/src/intel/tools/aubinator.c b/src/intel/tools/aubinator.c
index 9d29b68..89d29f2 100644
--- a/src/intel/tools/aubinator.c
+++ b/src/intel/tools/aubinator.c
@@ -91,18 +91,14 @@ print_dword_val(struct gen_field_iterator *iter, uint64_t offset,
                 int *dword_num)
 {
    struct gen_field *f;
-   union {
-      uint32_t dw;
-      float f;
-   } v;
 
    f = iter->group->fields[iter->i - 1];
-   v.dw = iter->p[f->start / 32];
+   const int dword = f->start / 32;
 
-   if (*dword_num != (f->start / 32)) {
+   if (*dword_num != dword) {
       printf("0x%08lx:  0x%08x : Dword %d\n",
-             offset + 4 * (f->start / 32), v.dw, f->start / 32);
-      *dword_num = (f->start / 32);
+             offset + 4 * dword,  iter->p[dword], dword);
+      *dword_num = dword;
    }
 }