2011/8/26 Georg-Johann Lay <a...@gjlay.de>: > Georg-Johann Lay wrote: >> progmem_section is a section to put jump tables in. >> >> This patch puts jump tables in individual sections if >> -ffunction-section is on and does some more cleanup around >> that, i.e. implement TARGET_ASM_FUNCTION_RODATA_SECTION hook. >> >> progmem_section is renamed to progmem_swtable_section and is >> local to avr.c now. >> >> What I don't understand is the old restriction that ASM_OUTPUT_ALIGN >> only printed .p2align for powers > 1; I changed that to >= 1 so that >> jump tables get aligned properly. >> >> Passed without regressions. > > This is an update of the patch. The differences to the original patch > http://gcc.gnu.org/ml/gcc-patches/2011-08/msg02090.html > are: > > * avr_asm_function_rodata_section now is based on > varasm.c:default_function_rodata_section and thus more general and > consistent. Basically, it replaces .rodata prefix with > .progmem.gcc_sw_table prefix etc. > > * ASM_OUTPUT_BEFORE_CASE_LABEL don't use ASM_OUTPUT_ALIGN to output the > alignment, and ASM_OUTPUT_ALIGN is unchanged. > > Passed without regressions. > > Actually, it's bit more than a cleanup now with the new > TARGET_ASM_FUNCTION_RODATA_SECTION... > > Ok to commit? >
Ok. Denis.
