summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authormmitchel <mmitchel@138bc75d-0d04-0410-961f-82ee72b054a4>1999-10-24 04:08:16 +0000
committermmitchel <mmitchel@138bc75d-0d04-0410-961f-82ee72b054a4>1999-10-24 04:08:16 +0000
commit8e658c4267773c7ffd2503ecc2b4fc2e89a34457 (patch)
tree0e06e50fb911e6ce7c9899a592714185e2593146
parentbd5b9fcb88e0c1b92aed6be578786539065e9b8d (diff)
downloadppe42-gcc-8e658c4267773c7ffd2503ecc2b4fc2e89a34457.tar.gz
ppe42-gcc-8e658c4267773c7ffd2503ecc2b4fc2e89a34457.zip
* integrate.c (integrate_decl_tree): Tweak setting of DECL_CONTEXT
for inlined declarations. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@30140 138bc75d-0d04-0410-961f-82ee72b054a4
-rw-r--r--gcc/ChangeLog5
-rw-r--r--gcc/integrate.c34
2 files changed, 22 insertions, 17 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index de9003a73fd..6192c8ecb26 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+Sat Oct 23 21:13:00 1999 Mark Mitchell <mark@codesourcery.com>
+
+ * integrate.c (integrate_decl_tree): Tweak setting of DECL_CONTEXT
+ for inlined declarations.
+
Fri Oct 22 18:05:43 1999 Jeffrey A Law (law@cygnus.com)
* arm.c (logical_binary_operator): New fucntion.
diff --git a/gcc/integrate.c b/gcc/integrate.c
index e312b188531..6c5abe8c1be 100644
--- a/gcc/integrate.c
+++ b/gcc/integrate.c
@@ -1419,23 +1419,23 @@ integrate_decl_tree (let, map)
if (DECL_LANG_SPECIFIC (d))
copy_lang_decl (d);
- /* ??? We used to call pushdecl here, but that does not work now that
- we generate entire functions as trees. We only want the pushdecl
- code that sets DECL_CONTEXT. Each front end sets DECL_CONTEXT
- slightly differently though, so we may need new callbacks to the
- front-ends to do this right. For now, we just use the code from the
- C front end and hope that is sufficient. Alternatively, we could
- set DECL_CONTEXT (d) here only if DECL_CONTEXT (t) is non-null. */
- /* This new declaration is now in the scope of the function into
- which we are inlining the function, not the function being
- inlined. */
- DECL_CONTEXT (d) = current_function_decl;
- /* A local extern declaration for a function doesn't constitute nesting.
- A local auto declaration does, since it's a forward decl
- for a nested function coming later. */
- if (TREE_CODE (d) == FUNCTION_DECL && DECL_INITIAL (d) == 0
- && DECL_EXTERNAL (d))
- DECL_CONTEXT (d) = 0;
+ /* Set the context for the new declaration. */
+ if (!DECL_CONTEXT (t))
+ /* Globals stay global. */
+ ;
+ else if (DECL_CONTEXT (t) != map->fndecl)
+ /* Things that weren't in the scope of the function we're
+ inlining from aren't in the scope we're inlining too,
+ either. */
+ ;
+ else if (TREE_STATIC (t))
+ /* Function-scoped static variables should say in the original
+ function. */
+ ;
+ else
+ /* Ordinary automatic local variables are now in the scope of
+ the new function. */
+ DECL_CONTEXT (d) = current_function_decl;
/* Add this declaration to the list of variables in the new
block. */
OpenPOWER on IntegriCloud