diff options
| author | Richard Guenther <rguenther@suse.de> | 2011-04-14 12:19:28 +0000 |
|---|---|---|
| committer | Richard Biener <rguenth@gcc.gnu.org> | 2011-04-14 12:19:28 +0000 |
| commit | ab4472fa10153d18b16704086d28599adae30e28 (patch) | |
| tree | 37469c49ec7265d49916385efd0f288cc4709c98 | |
| parent | 32dabdaff895ad60c48e5ebc2e8becedc0fd17d4 (diff) | |
| download | gcc-ab4472fa10153d18b16704086d28599adae30e28.zip gcc-ab4472fa10153d18b16704086d28599adae30e28.tar.gz gcc-ab4472fa10153d18b16704086d28599adae30e28.tar.bz2 | |
re PR middle-end/48590 (Alias analysis confused by builtin-stack-save/restore)
2011-04-14 Richard Guenther <rguenther@suse.de>
PR tree-optimization/48590
* tree-ssa-alias.c (ref_maybe_used_by_call_p_1): Handle
BUILT_IN_ALLOCA, BUILT_IN_STACK_SAVE and BUILT_IN_STACK_RESTORE.
(call_may_clobber_ref_p_1): Handle BUILT_IN_ALLOCA and
BUILT_IN_STACK_SAVE.
* tree-ssa-dce.c (propagate_necessity): Handle
BUILT_IN_ALLOCA, BUILT_IN_STACK_SAVE and BUILT_IN_STACK_RESTORE.
From-SVN: r172427
| -rw-r--r-- | gcc/ChangeLog | 10 | ||||
| -rw-r--r-- | gcc/tree-ssa-alias.c | 6 | ||||
| -rw-r--r-- | gcc/tree-ssa-dce.c | 5 |
3 files changed, 20 insertions, 1 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index d6ce73141..40fdfea 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,13 @@ +2011-04-14 Richard Guenther <rguenther@suse.de> + + PR tree-optimization/48590 + * tree-ssa-alias.c (ref_maybe_used_by_call_p_1): Handle + BUILT_IN_ALLOCA, BUILT_IN_STACK_SAVE and BUILT_IN_STACK_RESTORE. + (call_may_clobber_ref_p_1): Handle BUILT_IN_ALLOCA and + BUILT_IN_STACK_SAVE. + * tree-ssa-dce.c (propagate_necessity): Handle + BUILT_IN_ALLOCA, BUILT_IN_STACK_SAVE and BUILT_IN_STACK_RESTORE. + 2011-04-14 Nicola Pero <nicola.pero@meta-innovation.com> * c-parser.c (c_parser_objc_class_declaration): Updated call to diff --git a/gcc/tree-ssa-alias.c b/gcc/tree-ssa-alias.c index 4edacb5..1e80f49 100644 --- a/gcc/tree-ssa-alias.c +++ b/gcc/tree-ssa-alias.c @@ -1196,6 +1196,9 @@ ref_maybe_used_by_call_p_1 (gimple call, ao_ref *ref) case BUILT_IN_FREE: case BUILT_IN_MALLOC: case BUILT_IN_CALLOC: + case BUILT_IN_ALLOCA: + case BUILT_IN_STACK_SAVE: + case BUILT_IN_STACK_RESTORE: case BUILT_IN_MEMSET: case BUILT_IN_FREXP: case BUILT_IN_FREXPF: @@ -1432,6 +1435,9 @@ call_may_clobber_ref_p_1 (gimple call, ao_ref *ref) && targetm.ref_may_alias_errno (ref)) return true; return false; + case BUILT_IN_STACK_SAVE: + case BUILT_IN_ALLOCA: + return false; /* Freeing memory kills the pointed-to memory. More importantly the call has to serve as a barrier for moving loads and stores across it. */ diff --git a/gcc/tree-ssa-dce.c b/gcc/tree-ssa-dce.c index c46836da..6dc8a57 100644 --- a/gcc/tree-ssa-dce.c +++ b/gcc/tree-ssa-dce.c @@ -824,7 +824,10 @@ propagate_necessity (struct edge_list *el) && DECL_BUILT_IN_CLASS (callee) == BUILT_IN_NORMAL && (DECL_FUNCTION_CODE (callee) == BUILT_IN_MEMSET || DECL_FUNCTION_CODE (callee) == BUILT_IN_MALLOC - || DECL_FUNCTION_CODE (callee) == BUILT_IN_FREE)) + || DECL_FUNCTION_CODE (callee) == BUILT_IN_FREE + || DECL_FUNCTION_CODE (callee) == BUILT_IN_ALLOCA + || DECL_FUNCTION_CODE (callee) == BUILT_IN_STACK_SAVE + || DECL_FUNCTION_CODE (callee) == BUILT_IN_STACK_RESTORE)) continue; /* Calls implicitly load from memory, their arguments |