Re: [patch] For alpha-vms, unset flag_jump_tables if flag_pic is nonzero

2012-05-16 Thread Paul_Koning
On May 16, 2012, at 1:27 PM, wrote: > It turns out that there is a second target that doesn't define > ASM_OUTPUT_ADDR_DIFF_ELT: pdp11. I'll fix that. > > The documentation is out of date as a consequence of this patch, because it > still says that ASM_OUTPUT_ADDR_DIFF_ELT is optional whic

Re: [patch] For alpha-vms, unset flag_jump_tables if flag_pic is nonzero

2012-05-16 Thread Paul_Koning
It turns out that there is a second target that doesn't define ASM_OUTPUT_ADDR_DIFF_ELT: pdp11. I'll fix that. The documentation is out of date as a consequence of this patch, because it still says that ASM_OUTPUT_ADDR_DIFF_ELT is optional which is no longer the case (or not unless you take so

Re: [patch] For alpha-vms, unset flag_jump_tables if flag_pic is nonzero

2012-05-03 Thread Tristan Gingold
On May 2, 2012, at 9:12 PM, Steven Bosscher wrote: > Hello, > > alpha-vms is the only target that does not define > ASM_OUTPUT_ADDR_DIFF_ELT. That makes the code in stmt.c to handle this > case alpha-vms specific. But there is a better way to handle this: > Just mimic -fno-jump-tables if flag_pi

Re: [patch] For alpha-vms, unset flag_jump_tables if flag_pic is nonzero

2012-05-03 Thread Tristan Gingold
On May 2, 2012, at 11:45 PM, Richard Henderson wrote: > On 05/02/2012 12:26 PM, Steven Bosscher wrote: >>> * config/alpha/vms.h (SUBTARGET_OVERRIDE_OPTIONS): For pic code, >>> unset flag_jump_tables. >>> * stmt.c (expand_case): Remove special flag_pic case conditional >>>

Re: [patch] For alpha-vms, unset flag_jump_tables if flag_pic is nonzero

2012-05-02 Thread Richard Henderson
On 05/02/2012 12:26 PM, Steven Bosscher wrote: * config/alpha/vms.h (SUBTARGET_OVERRIDE_OPTIONS): For pic code, unset flag_jump_tables. * stmt.c (expand_case): Remove special flag_pic case conditional on ASM_OUTPUT_ADDR_DIFF_ELT not being defined. Ok. I had

Re: [patch] For alpha-vms, unset flag_jump_tables if flag_pic is nonzero

2012-05-02 Thread Steven Bosscher
On Wed, May 2, 2012 at 9:12 PM, Steven Bosscher wrote: > Hello, > > alpha-vms is the only target that does not define > ASM_OUTPUT_ADDR_DIFF_ELT. That makes the code in stmt.c to handle this > case alpha-vms specific. But there is a better way to handle this: > Just mimic -fno-jump-tables if flag_

[patch] For alpha-vms, unset flag_jump_tables if flag_pic is nonzero

2012-05-02 Thread Steven Bosscher
Hello, alpha-vms is the only target that does not define ASM_OUTPUT_ADDR_DIFF_ELT. That makes the code in stmt.c to handle this case alpha-vms specific. But there is a better way to handle this: Just mimic -fno-jump-tables if flag_pic is nonzero. Tested by building a cross-compiler for alpha-dec-