On 07/20/2011 02:00 PM, H.J. Lu wrote: > operands[0] = expand_simple_binop (Pmode, code, op0, op1, NULL_RTX, 0, > OPTAB_DIRECT); > } > + else if (TARGET_X32) > + operands[0] = convert_memory_address (Pmode, operands[0]); > }) > > (define_insn "*tablejump_1" > --- > > Richard, is this OK for trunk with the last patch if they work?
Yeah. I had put the convert_memory_address first, because I thought it would help match modes inside the PIC section as well. Anyway, I see that expand_simple_binop will handle mode conversions itself, so this placement is also ok. I'm only mildly curious as to why that placement failed. r~