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~

Reply via email to