On 03/21/2016 11:44 AM, Richard Henderson wrote:
On 03/21/2016 06:40 AM, Jiong Wang wrote:
On 17/03/16 19:17, Richard Henderson wrote:
PR target/70120
    * varasm.c (for_each_section): New.
    * varasm.h (for_each_section): Declare.
    * config/aarch64/aarch64.c (aarch64_align_code_section): New.
    (aarch64_asm_file_end): New.
    (TARGET_ASM_FILE_END): Redefine.

Will ASM_OUTPUT_POOL_EPILOGUE be a better place to fix this issue?
which can
avoid the for_each_section traversal.

It's a good point.  I hadn't noticed this old (and currently unused) hook.
This alternate patch does in fact work.


r~

z


        * config/aarch64/aarch64.c (aarch64_asm_output_pool_epilogue): New.
        * config/aarch64/aarch64-protos.h: Declare it.
        * config/aarch64/aarch64.h (ASM_OUTPUT_POOL_EPILOGUE): New.
Approved and installed (along with the 3 testcases from the original submission.

Thanks,
Jeff

Reply via email to