On Wed, Jul 25, 2012 at 11:11 AM, Sandra Loosemore
<[email protected]> wrote:
> On 07/25/2012 09:57 AM, Richard Henderson wrote:
>>
>> 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.
>
> Thanks! Here's the version I committed.
>
> -Sandra
>
>
> 2012-07-25 Sandra Loosemore <[email protected]>
> Paul Brook <[email protected]>
>
> 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.
You didn't mention that you included "target.h" nor add "target.h"
dependency on tree-cfg.c. I am checking in this as an obvious fix.
--
H.J.
--
diff --git a/gcc/Makefile.in b/gcc/Makefile.in
index 65a49d7..f2366ad 100644
--- a/gcc/Makefile.in
+++ b/gcc/Makefile.in
@@ -2363,7 +2363,7 @@ tree-vrp.o : tree-vrp.c $(CONFIG_H) $(SYSTEM_H)
coretypes.h $(TM_H) $(TREE_H) \
$(CFGLOOP_H) $(SCEV_H) intl.h \
$(GIMPLE_PRETTY_PRINT_H) gimple-fold.h $(OPTABS_H) $(EXPR_H)
tree-cfg.o : tree-cfg.c $(TREE_FLOW_H) $(CONFIG_H) $(SYSTEM_H) \
- $(TREE_H) $(TM_P_H) $(GGC_H) $(FLAGS_H) \
+ $(TREE_H) $(TM_P_H) $(GGC_H) $(FLAGS_H) $(TARGET_H) \
$(DIAGNOSTIC_CORE_H) $(FUNCTION_H) $(TM_H) coretypes.h \
$(TREE_DUMP_H) $(EXCEPT_H) $(CFGLOOP_H) $(TREE_PASS_H) \
$(BASIC_BLOCK_H) \