On 07/24/2012 09:40 PM, Sandra Loosemore wrote:
> PR target/53633
>
> gcc/
> * target.def (warn_func_return): New hook.
> * doc/tm.texi.in (TARGET_WARN_FUNC_RETURN): New hook.
> * doc/tm.texi: Regenerate.
> * doc/sourcebuild.texi (Effective-Target Keywords): Document
> naked_functions.
> * ipa-pure-const.c (warn_function_noreturn): Check
> targetm.warn_func_return.
> * tree-cfg.c (execute_warn_function_return): Likewise.
> * config/spu/spu.c (spu_warn_func_return): New.
> (TARGET_WARN_FUNC_RETURN): Define.
> * config/rx/rx.c (rx_warn_func_return): New.
> (TARGET_WARN_FUNC_RETURN): Define.
> * config/avr/avr.c (avr_warn_func_return): New.
> (TARGET_WARN_FUNC_RETURN): Define.
> * config/arm/arm.c (arm_warn_func_return): New.
> (TARGET_WARN_FUNC_RETURN): Define.
> * config/mcore/mcore.c (mcore_warn_func_return): New.
> (TARGET_WARN_FUNC_RETURN): Define.
> (saved_warn_return_type, saved_warn_return_type_count): Remove.
> (mcore_reorg, mcore_handle_naked_attribute): Remove warn_return hack.
>
> gcc/cp/
> * decl.c (finish_function): Check targetm.warn_func_return.
>
> gcc/testsuite/
> * lib/target-suports.exp (check_effective_target_naked_functions):
> New.
> * c-c++-common/pr53633.c: New test.
I'll echo Nick's comments about arm asm in a common test.
There's no need to have anything but __asm__(""); there.
Ok with that change.
r~