On Fri, Mar 16, 2018 at 07:01:18PM -0400, Michael Meissner wrote:
> In patch #4, I mentioned that the spec 2006 benchmark 'tonto' generated
> different with the patches applied.  I tracked it down, and it was due to the
> call I inserted in rs6000_debug_reg_print to update the conditional register
> usage seemed to set the Altivec registers VS0..VS19 to call_used instead of
> call_saved.  Since I no longer need to set the conditional register usage with
> -mdebug=reg, it is simpler to just delete it.
> 
> 2018-03-16  Michael Meissner  <meiss...@linux.vnet.ibm.com>
> 
>       * config/rs6000/rs6000.c (rs6000_debug_reg_print): Eliminate call
>       to rs6000_conditional_register_usage.

Yes, debug output should *never* change *any* state.

Could you fold this patch into the patch that created the problem please?


Segher

Reply via email to