Re: RFA: Replace LAST_INSN_CODE with NUM_INSN_CODES

2015-08-21 Thread Jeff Law
On 08/21/2015 08:36 AM, Richard Sandiford wrote: Richard Sandiford writes: Claudiu reported that I'd botched the definition of LAST_INSN_CODE in my recent patches to reduce the size of insn_data. I'd defined it as the last valid insn code, whereas it's supposed to be the last valid code +1. T

Re: RFA: Replace LAST_INSN_CODE with NUM_INSN_CODES

2015-08-21 Thread Richard Sandiford
Richard Sandiford writes: > Claudiu reported that I'd botched the definition of LAST_INSN_CODE > in my recent patches to reduce the size of insn_data. I'd defined > it as the last valid insn code, whereas it's supposed to be the > last valid code +1. > > This patch replaces LAST_INSN_CODE with a

RFA: Replace LAST_INSN_CODE with NUM_INSN_CODES

2015-08-21 Thread Richard Sandiford
Claudiu reported that I'd botched the definition of LAST_INSN_CODE in my recent patches to reduce the size of insn_data. I'd defined it as the last valid insn code, whereas it's supposed to be the last valid code +1. This patch replaces LAST_INSN_CODE with a separate NUM_INSN_CODES count, outside