No point in duplicating the shift logic when we have macros already to
keep that all unified.
+2015-06-11  Mike Frysinger  <vapier@gentoo.org>
+
+       * sim-trace.h (TRACE_USEFUL_MASK): Change to TRACE_xxx defines.
+
 2015-06-11  Mike Frysinger  <vapier@gentoo.org>
 
        * sim-trace.h (WITH_TRACE_P): New define.
 
    splat on the screen everything under the sun making nothing easy to
    find.  */
 #define TRACE_USEFUL_MASK \
-((1 << TRACE_INSN_IDX) \
- | (1 << TRACE_LINENUM_IDX) \
- | (1 << TRACE_MEMORY_IDX) \
- | (1 << TRACE_MODEL_IDX))
+  (TRACE_insn | TRACE_linenum | TRACE_memory | TRACE_model)
 \f
 /* Masks so WITH_TRACE can have symbolic values.
    The case choice here is on purpose.  The lowercase parts are args to