X-Git-Url: http://git.sourceforge.jp/view?a=blobdiff_plain;f=gcc%2Fcalls.c;h=27adf1457e818c7d2803f53f866596dc339ae3ce;hb=f12b58b3384f7b4faba39eaa3ebc67c3049f2a3c;hp=3561f5987f40d62ade43527b51b62dee9d01353c;hpb=a9f2963b8b4bdedeb77ea791c4b81205cc8b4832;p=pf3gnuchains%2Fgcc-fork.git diff --git a/gcc/calls.c b/gcc/calls.c index 3561f5987f4..27adf1457e8 100644 --- a/gcc/calls.c +++ b/gcc/calls.c @@ -1,23 +1,23 @@ /* Convert function calls to rtl insns, for GNU C compiler. Copyright (C) 1989, 1992, 1993, 1994, 1995, 1996, 1997, 1998 - 1999, 2000 Free Software Foundation, Inc. + 1999, 2000, 2001 Free Software Foundation, Inc. -This file is part of GNU CC. +This file is part of GCC. -GNU CC is free software; you can redistribute it and/or modify -it under the terms of the GNU General Public License as published by -the Free Software Foundation; either version 2, or (at your option) -any later version. +GCC is free software; you can redistribute it and/or modify it under +the terms of the GNU General Public License as published by the Free +Software Foundation; either version 2, or (at your option) any later +version. -GNU CC is distributed in the hope that it will be useful, -but WITHOUT ANY WARRANTY; without even the implied warranty of -MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the -GNU General Public License for more details. +GCC is distributed in the hope that it will be useful, but WITHOUT ANY +WARRANTY; without even the implied warranty of MERCHANTABILITY or +FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License +for more details. You should have received a copy of the GNU General Public License -along with GNU CC; see the file COPYING. If not, write to -the Free Software Foundation, 59 Temple Place - Suite 330, -Boston, MA 02111-1307, USA. */ +along with GCC; see the file COPYING. If not, write to the Free +Software Foundation, 59 Temple Place - Suite 330, Boston, MA +02111-1307, USA. */ #include "config.h" #include "system.h" @@ -25,35 +25,19 @@ Boston, MA 02111-1307, USA. */ #include "tree.h" #include "flags.h" #include "expr.h" +#include "libfuncs.h" #include "function.h" #include "regs.h" -#include "insn-flags.h" #include "toplev.h" #include "output.h" #include "tm_p.h" #include "timevar.h" - -#ifndef ACCUMULATE_OUTGOING_ARGS -#define ACCUMULATE_OUTGOING_ARGS 0 -#endif - -/* Supply a default definition for PUSH_ARGS. */ -#ifndef PUSH_ARGS -#ifdef PUSH_ROUNDING -#define PUSH_ARGS !ACCUMULATE_OUTGOING_ARGS -#else -#define PUSH_ARGS 0 -#endif -#endif +#include "sbitmap.h" #if !defined FUNCTION_OK_FOR_SIBCALL #define FUNCTION_OK_FOR_SIBCALL(DECL) 1 #endif -#if !defined PREFERRED_STACK_BOUNDARY && defined STACK_BOUNDARY -#define PREFERRED_STACK_BOUNDARY STACK_BOUNDARY -#endif - /* Decide whether a function's arguments should be processed from first to last or from last to first. @@ -72,6 +56,10 @@ Boston, MA 02111-1307, USA. */ #define PUSH_ARGS_REVERSED 0 #endif +#ifndef STACK_POINTER_OFFSET +#define STACK_POINTER_OFFSET 0 +#endif + /* Like PREFERRED_STACK_BOUNDARY but in units of bytes, not bits. */ #define STACK_BYTES (PREFERRED_STACK_BOUNDARY / BITS_PER_UNIT) @@ -145,6 +133,13 @@ static char *stack_usage_map; /* Size of STACK_USAGE_MAP. */ static int highest_outgoing_arg_in_use; +/* A bitmap of virtual-incoming stack space. Bit is set if the corresponding + stack location's tail call argument has been already stored into the stack. + This bitmap is used to prevent sibling call optimization if function tries + to use parent's incoming argument slots when they have been already + overwritten with tail call arguments. */ +static sbitmap stored_args_map; + /* stack_arg_under_construction is nonzero when an argument may be initialized with a constructor call (including a C function that returns a BLKmode struct) and expand_call must take special action @@ -155,11 +150,11 @@ int stack_arg_under_construction; static int calls_function PARAMS ((tree, int)); static int calls_function_1 PARAMS ((tree, int)); -/* Nonzero if this is a call to a `const' function. */ +/* Nonzero if this is a call to a `const' function. */ #define ECF_CONST 1 /* Nonzero if this is a call to a `volatile' function. */ #define ECF_NORETURN 2 -/* Nonzero if this is a call to malloc or a related function. */ +/* Nonzero if this is a call to malloc or a related function. */ #define ECF_MALLOC 4 /* Nonzero if it is plausible that this is a call to alloca. */ #define ECF_MAY_BE_ALLOCA 8 @@ -176,6 +171,11 @@ static int calls_function_1 PARAMS ((tree, int)); /* Nonzero if this is a call to "pure" function (like const function, but may read memory. */ #define ECF_PURE 512 +/* Nonzero if this is a call to a function that returns with the stack + pointer depressed. */ +#define ECF_SP_DEPRESSED 1024 +/* Nonzero if this call is known to always return. */ +#define ECF_ALWAYS_RETURN 2048 static void emit_call_1 PARAMS ((rtx, tree, tree, HOST_WIDE_INT, HOST_WIDE_INT, HOST_WIDE_INT, rtx, @@ -183,7 +183,7 @@ static void emit_call_1 PARAMS ((rtx, tree, tree, HOST_WIDE_INT, static void precompute_register_parameters PARAMS ((int, struct arg_data *, int *)); -static void store_one_arg PARAMS ((struct arg_data *, rtx, int, int, +static int store_one_arg PARAMS ((struct arg_data *, rtx, int, int, int)); static void store_unaligned_arguments_into_pseudos PARAMS ((struct arg_data *, int)); @@ -192,7 +192,7 @@ static int finalize_must_preallocate PARAMS ((int, int, struct args_size *)); static void precompute_arguments PARAMS ((int, int, struct arg_data *)); -static int compute_argument_block_size PARAMS ((int, +static int compute_argument_block_size PARAMS ((int, struct args_size *, int)); static void initialize_argument_information PARAMS ((int, @@ -207,14 +207,17 @@ static void compute_argument_addresses PARAMS ((struct arg_data *, static rtx rtx_for_function_call PARAMS ((tree, tree)); static void load_register_parameters PARAMS ((struct arg_data *, int, rtx *, int)); -static int libfunc_nothrow PARAMS ((rtx)); -static rtx emit_library_call_value_1 PARAMS ((int, rtx, rtx, int, +static rtx emit_library_call_value_1 PARAMS ((int, rtx, rtx, + enum libcall_type, enum machine_mode, int, va_list)); static int special_function_p PARAMS ((tree, int)); static int flags_from_decl_or_type PARAMS ((tree)); static rtx try_to_integrate PARAMS ((tree, tree, rtx, int, tree, rtx)); +static int check_sibcall_argument_overlap_1 PARAMS ((rtx)); +static int check_sibcall_argument_overlap PARAMS ((rtx, struct arg_data *)); + static int combine_pending_stack_adjustment_and_call PARAMS ((int, struct args_size *, int)); @@ -267,6 +270,11 @@ calls_function_1 (exp, which) case CALL_EXPR: if (which == 0) return 1; + else if ((TREE_CODE (TREE_TYPE (TREE_TYPE (TREE_OPERAND (exp, 0)))) + == FUNCTION_TYPE) + && (TYPE_RETURNS_STACK_DEPRESSED + (TREE_TYPE (TREE_TYPE (TREE_OPERAND (exp, 0)))))) + return 1; else if (TREE_CODE (TREE_OPERAND (exp, 0)) == ADDR_EXPR && (TREE_CODE (TREE_OPERAND (TREE_OPERAND (exp, 0), 0)) == FUNCTION_DECL) @@ -277,6 +285,17 @@ calls_function_1 (exp, which) break; + case CONSTRUCTOR: + { + tree tem; + + for (tem = CONSTRUCTOR_ELTS (exp); tem != 0; tem = TREE_CHAIN (tem)) + if (calls_function_1 (TREE_VALUE (tem), which)) + return 1; + } + + return 0; + case SAVE_EXPR: if (SAVE_EXPR_RTL (exp) != 0) return 0; @@ -335,11 +354,12 @@ calls_function_1 (exp, which) CALL_INSN_FUNCTION_USAGE information. */ rtx -prepare_call_address (funexp, fndecl, call_fusage, reg_parm_seen) +prepare_call_address (funexp, fndecl, call_fusage, reg_parm_seen, sibcallp) rtx funexp; tree fndecl; rtx *call_fusage; int reg_parm_seen; + int sibcallp; { rtx static_chain_value = 0; @@ -357,7 +377,7 @@ prepare_call_address (funexp, fndecl, call_fusage, reg_parm_seen) funexp = ((SMALL_REGISTER_CLASSES && reg_parm_seen) ? force_not_mem (memory_address (FUNCTION_MODE, funexp)) : memory_address (FUNCTION_MODE, funexp)); - else + else if (! sibcallp) { #ifndef NO_FUNCTION_CSE if (optimize && ! flag_no_function_cse) @@ -417,7 +437,7 @@ prepare_call_address (funexp, fndecl, call_fusage, reg_parm_seen) We restore `inhibit_defer_pop' to that value. CALL_FUSAGE is either empty or an EXPR_LIST of USE expressions that - denote registers used by the called function. */ + denote registers used by the called function. */ static void emit_call_1 (funexp, fndecl, funtype, stack_size, rounded_stack_size, @@ -453,10 +473,9 @@ emit_call_1 (funexp, fndecl, funtype, stack_size, rounded_stack_size, #if defined (HAVE_sibcall_pop) && defined (HAVE_sibcall_value_pop) if ((ecf_flags & ECF_SIBCALL) && HAVE_sibcall_pop && HAVE_sibcall_value_pop - && (RETURN_POPS_ARGS (fndecl, funtype, stack_size) > 0 - || stack_size == 0)) + && (n_popped > 0 || stack_size == 0)) { - rtx n_pop = GEN_INT (RETURN_POPS_ARGS (fndecl, funtype, stack_size)); + rtx n_pop = GEN_INT (n_popped)); rtx pat; /* If this subroutine pops its own args, record that in the call insn @@ -484,7 +503,7 @@ emit_call_1 (funexp, fndecl, funtype, stack_size, rounded_stack_size, even if the call has no arguments to pop. */ #if defined (HAVE_call) && defined (HAVE_call_value) if (HAVE_call && HAVE_call_value && HAVE_call_pop && HAVE_call_value_pop - && n_popped > 0) + && n_popped > 0 && ! (ecf_flags & ECF_SP_DEPRESSED)) #else if (HAVE_call_pop && HAVE_call_value_pop) #endif @@ -578,7 +597,7 @@ emit_call_1 (funexp, fndecl, funtype, stack_size, rounded_stack_size, /* If this is a const call, then set the insn's unchanging bit. */ if (ecf_flags & (ECF_CONST | ECF_PURE)) - CONST_CALL_P (call_insn) = 1; + CONST_OR_PURE_CALL_P (call_insn) = 1; /* If this call can't throw, attach a REG_EH_REGION reg note to that effect. */ @@ -586,6 +605,17 @@ emit_call_1 (funexp, fndecl, funtype, stack_size, rounded_stack_size, REG_NOTES (call_insn) = gen_rtx_EXPR_LIST (REG_EH_REGION, const0_rtx, REG_NOTES (call_insn)); + if (ecf_flags & ECF_NORETURN) + REG_NOTES (call_insn) = gen_rtx_EXPR_LIST (REG_NORETURN, const0_rtx, + REG_NOTES (call_insn)); + if (ecf_flags & ECF_ALWAYS_RETURN) + REG_NOTES (call_insn) = gen_rtx_EXPR_LIST (REG_ALWAYS_RETURN, const0_rtx, + REG_NOTES (call_insn)); + + if (ecf_flags & ECF_RETURNS_TWICE) + REG_NOTES (call_insn) = gen_rtx_EXPR_LIST (REG_SETJMP, const0_rtx, + REG_NOTES (call_insn)); + SIBLING_CALL_P (call_insn) = ((ecf_flags & ECF_SIBCALL) != 0); /* Restore this now, so that we do defer pops for this call's args @@ -613,10 +643,10 @@ emit_call_1 (funexp, fndecl, funtype, stack_size, rounded_stack_size, If returning from the subroutine does pop the args, indicate that the stack pointer will be changed. */ - if (rounded_stack_size != 0) + if (rounded_stack_size != 0 && ! (ecf_flags & ECF_SP_DEPRESSED)) { if (flag_defer_pop && inhibit_defer_pop == 0 - && !(ecf_flags & (ECF_CONST | ECF_PURE))) + && ! (ecf_flags & (ECF_CONST | ECF_PURE))) pending_stack_adjust += rounded_stack_size; else adjust_stack (rounded_stack_size_rtx); @@ -630,7 +660,7 @@ emit_call_1 (funexp, fndecl, funtype, stack_size, rounded_stack_size, ??? We may optimize similar to defer_pop above, but it is probably not worthwhile. - + ??? It will be worthwhile to enable combine_stack_adjustments even for such machines. */ else if (n_popped) @@ -664,8 +694,8 @@ special_function_p (fndecl, flags) think they are. */ && DECL_CONTEXT (fndecl) == NULL_TREE && TREE_PUBLIC (fndecl)) { - char *name = IDENTIFIER_POINTER (DECL_NAME (fndecl)); - char *tname = name; + const char *name = IDENTIFIER_POINTER (DECL_NAME (fndecl)); + const char *tname = name; /* We assume that alloca will always be called by name. It makes no sense to pass it as a pointer-to-function to @@ -734,7 +764,7 @@ special_function_p (fndecl, flags) it may return the same address across multiple calls. C++ operator new is not suitable because it is not required to return a unique pointer; indeed, the standard placement new - just returns its argument. */ + just returns its argument. */ else if (TYPE_MODE (TREE_TYPE (TREE_TYPE (fndecl))) == Pmode && (! strcmp (tname, "malloc") || ! strcmp (tname, "calloc") @@ -745,6 +775,7 @@ special_function_p (fndecl, flags) } /* Return nonzero when tree represent call to longjmp. */ + int setjmp_call_p (fndecl) tree fndecl; @@ -753,11 +784,13 @@ setjmp_call_p (fndecl) } /* Detect flags (function attributes) from the function type node. */ + static int flags_from_decl_or_type (exp) tree exp; { int flags = 0; + /* ??? We can't set IS_MALLOC for function types? */ if (DECL_P (exp)) { @@ -773,7 +806,7 @@ flags_from_decl_or_type (exp) flags |= ECF_NOTHROW; } - if (TREE_READONLY (exp) && !TREE_THIS_VOLATILE (exp)) + if (TREE_READONLY (exp) && ! TREE_THIS_VOLATILE (exp)) flags |= ECF_CONST; if (TREE_THIS_VOLATILE (exp)) @@ -782,7 +815,6 @@ flags_from_decl_or_type (exp) return flags; } - /* Precompute all register parameters as described by ARGS, storing values into fields within the ARGS array. @@ -827,7 +859,7 @@ precompute_register_parameters (num_actuals, args, reg_parm_seen) TYPE_MODE (TREE_TYPE (args[i].tree_value)), args[i].value, args[i].unsignedp); - /* If the value is expensive, and we are inside an appropriately + /* If the value is expensive, and we are inside an appropriately short loop, put the value into a pseudo and then put the pseudo into the hard reg. @@ -839,7 +871,7 @@ precompute_register_parameters (num_actuals, args, reg_parm_seen) || (GET_CODE (args[i].value) == SUBREG && GET_CODE (SUBREG_REG (args[i].value)) == REG))) && args[i].mode != BLKmode - && rtx_cost (args[i].value, SET) > 2 + && rtx_cost (args[i].value, SET) > COSTS_N_INSNS (1) && ((SMALL_REGISTER_CLASSES && *reg_parm_seen) || preserve_subexpressions_p ())) args[i].value = copy_to_mode_reg (args[i].mode, args[i].value); @@ -889,7 +921,7 @@ save_fixed_argument_area (reg_parm_stack_space, argblock, /* If we don't have the required alignment, must do this in BLKmode. */ if ((*low_to_save & (MIN (GET_MODE_SIZE (save_mode), - BIGGEST_ALIGNMENT / UNITS_PER_WORD) - 1))) + BIGGEST_ALIGNMENT / UNITS_PER_WORD) - 1))) save_mode = BLKmode; #ifdef ARGS_GROW_DOWNWARD @@ -953,12 +985,12 @@ restore_fixed_argument_area (save_area, argblock, high_to_save, low_to_save) move_by_pieces (stack_area, validize_mem (save_area), high_to_save - low_to_save + 1, PARM_BOUNDARY); } -#endif - +#endif /* REG_PARM_STACK_SPACE */ + /* If any elements in ARGS refer to parameters that are to be passed in registers, but not in memory, and whose alignment does not permit a direct copy into registers. Copy the values into a group of pseudos - which we will later copy into the appropriate hard registers. + which we will later copy into the appropriate hard registers. Pseudos for each unaligned argument will be stored into the array args[argnum].aligned_regs. The caller is responsible for deallocating @@ -970,7 +1002,7 @@ store_unaligned_arguments_into_pseudos (args, num_actuals) int num_actuals; { int i, j; - + for (i = 0; i < num_actuals; i++) if (args[i].reg != 0 && ! args[i].pass_on_stack && args[i].mode == BLKmode @@ -1027,7 +1059,7 @@ store_unaligned_arguments_into_pseudos (args, num_actuals) } /* Fill in ARGS_SIZE and ARGS array based on the parameters found in - ACTPARMS. + ACTPARMS. NUM_ACTUALS is the total number of parameters. @@ -1075,7 +1107,7 @@ initialize_argument_information (num_actuals, args, args_size, n_named_args, struct args_size alignment_pad; int i; tree p; - + args_size->constant = 0; args_size->var = 0; @@ -1158,8 +1190,8 @@ initialize_argument_information (num_actuals, args, args_size, n_named_args, but it is safe in the only case where this is a useful optimization; namely, when the argument is a plain object. In that case, the frontend is just asking the backend to - make a bitwise copy of the argument. */ - + make a bitwise copy of the argument. */ + if (TREE_CODE (args[i].tree_value) == TARGET_EXPR && (DECL_P (TREE_OPERAND (args[i].tree_value, 1))) && ! REG_P (DECL_RTL (TREE_OPERAND (args[i].tree_value, 1)))) @@ -1228,7 +1260,7 @@ initialize_argument_information (num_actuals, args, args_size, n_named_args, register window has to be unwinded before calling the routine, so arguments have to go into the incoming registers. */ args[i].tail_call_reg = FUNCTION_INCOMING_ARG (*args_so_far, mode, type, - argpos < n_named_args); + argpos < n_named_args); #else args[i].tail_call_reg = args[i].reg; #endif @@ -1288,7 +1320,7 @@ initialize_argument_information (num_actuals, args, args_size, n_named_args, args[i].size.constant -= ((args[i].partial * UNITS_PER_WORD) / (PARM_BOUNDARY / BITS_PER_UNIT) * (PARM_BOUNDARY / BITS_PER_UNIT)); - + /* Update ARGS_SIZE, the total stack space for args so far. */ args_size->constant += args[i].size.constant; @@ -1323,7 +1355,7 @@ initialize_argument_information (num_actuals, args, args_size, n_named_args, static int compute_argument_block_size (reg_parm_stack_space, args_size, - preferred_stack_boundary) + preferred_stack_boundary) int reg_parm_stack_space; struct args_size *args_size; int preferred_stack_boundary ATTRIBUTE_UNUSED; @@ -1333,10 +1365,8 @@ compute_argument_block_size (reg_parm_stack_space, args_size, /* For accumulate outgoing args mode we don't need to align, since the frame will be already aligned. Align to STACK_BOUNDARY in order to prevent backends from generating missaligned frame sizes. */ -#ifdef STACK_BOUNDARY if (ACCUMULATE_OUTGOING_ARGS && preferred_stack_boundary > STACK_BOUNDARY) preferred_stack_boundary = STACK_BOUNDARY; -#endif /* Compute the actual size of the argument block required. The variable and constant sizes must be combined, the size may have to be rounded, @@ -1347,18 +1377,16 @@ compute_argument_block_size (reg_parm_stack_space, args_size, args_size->var = ARGS_SIZE_TREE (*args_size); args_size->constant = 0; -#ifdef PREFERRED_STACK_BOUNDARY preferred_stack_boundary /= BITS_PER_UNIT; if (preferred_stack_boundary > 1) { /* We don't handle this case yet. To handle it correctly we have - to add the delta, round and substract the delta. + to add the delta, round and substract the delta. Currently no machine description requires this support. */ if (stack_pointer_delta & (preferred_stack_boundary - 1)) - abort(); + abort (); args_size->var = round_up (args_size->var, preferred_stack_boundary); } -#endif if (reg_parm_stack_space > 0) { @@ -1377,7 +1405,6 @@ compute_argument_block_size (reg_parm_stack_space, args_size, } else { -#ifdef PREFERRED_STACK_BOUNDARY preferred_stack_boundary /= BITS_PER_UNIT; if (preferred_stack_boundary < 1) preferred_stack_boundary = 1; @@ -1387,7 +1414,6 @@ compute_argument_block_size (reg_parm_stack_space, args_size, / preferred_stack_boundary * preferred_stack_boundary) - stack_pointer_delta); -#endif args_size->constant = MAX (args_size->constant, reg_parm_stack_space); @@ -1410,9 +1436,9 @@ compute_argument_block_size (reg_parm_stack_space, args_size, NUM_ACTUALS is the number of arguments. - ARGS is an array containing information for each argument; this routine - fills in the INITIAL_VALUE and VALUE fields for each precomputed argument. - */ + ARGS is an array containing information for each argument; this + routine fills in the INITIAL_VALUE and VALUE fields for each + precomputed argument. */ static void precompute_arguments (flags, num_actuals, args) @@ -1441,6 +1467,8 @@ precompute_arguments (flags, num_actuals, args) if ((flags & (ECF_CONST | ECF_PURE)) || calls_function (args[i].tree_value, !ACCUMULATE_OUTGOING_ARGS)) { + enum machine_mode mode; + /* If this is an addressable type, we cannot pre-evaluate it. */ if (TREE_ADDRESSABLE (TREE_TYPE (args[i].tree_value))) abort (); @@ -1460,11 +1488,11 @@ precompute_arguments (flags, num_actuals, args) args[i].initial_value = args[i].value = protect_from_queue (args[i].value, 0); - if (TYPE_MODE (TREE_TYPE (args[i].tree_value)) != args[i].mode) + mode = TYPE_MODE (TREE_TYPE (args[i].tree_value)); + if (mode != args[i].mode) { args[i].value - = convert_modes (args[i].mode, - TYPE_MODE (TREE_TYPE (args[i].tree_value)), + = convert_modes (args[i].mode, mode, args[i].value, args[i].unsignedp); #ifdef PROMOTE_FOR_CALL_ONLY /* CSE will replace this only if it contains args[i].value @@ -1474,8 +1502,7 @@ precompute_arguments (flags, num_actuals, args) && GET_MODE_CLASS (args[i].mode) == MODE_INT) { args[i].initial_value - = gen_rtx_SUBREG (TYPE_MODE (TREE_TYPE (args[i].tree_value)), - args[i].value, 0); + = gen_lowpart_SUBREG (mode, args[i].value); SUBREG_PROMOTED_VAR_P (args[i].initial_value) = 1; SUBREG_PROMOTED_UNSIGNED_P (args[i].initial_value) = args[i].unsignedp; @@ -1547,7 +1574,7 @@ finalize_must_preallocate (must_preallocate, num_actuals, args, args_size) /* If we preallocated stack space, compute the address of each argument and store it into the ARGS array. - We need not ensure it is a valid memory address here; it will be + We need not ensure it is a valid memory address here; it will be validized when it is used. ARGBLOCK is an rtx for the address of the outgoing arguments. */ @@ -1600,12 +1627,12 @@ compute_argument_addresses (args, argblock, num_actuals) outgoing arguments and we cannot allow reordering of reads from function arguments with stores to outgoing arguments of sibling calls. */ - MEM_ALIAS_SET (args[i].stack) = 0; - MEM_ALIAS_SET (args[i].stack_slot) = 0; + set_mem_alias_set (args[i].stack, 0); + set_mem_alias_set (args[i].stack_slot, 0); } } } - + /* Given a FNDECL and EXP, return an rtx suitable for use as a target address in a call instruction. @@ -1640,9 +1667,9 @@ rtx_for_function_call (fndecl, exp) { rtx funaddr; push_temp_slots (); - funaddr = funexp = - expand_expr (TREE_OPERAND (exp, 0), NULL_RTX, VOIDmode, 0); - pop_temp_slots (); /* FUNEXP can't be BLKmode */ + funaddr = funexp = + expand_expr (TREE_OPERAND (exp, 0), NULL_RTX, VOIDmode, 0); + pop_temp_slots (); /* FUNEXP can't be BLKmode. */ /* Check the function is executable. */ if (current_function_check_memory_usage) @@ -1650,12 +1677,11 @@ rtx_for_function_call (fndecl, exp) #ifdef POINTERS_EXTEND_UNSIGNED /* It might be OK to convert funexp in place, but there's a lot going on between here and when it happens naturally - that this seems safer. */ - funaddr = convert_memory_address (Pmode, funexp); + that this seems safer. */ + funaddr = convert_memory_address (Pmode, funexp); #endif - emit_library_call (chkr_check_exec_libfunc, 1, - VOIDmode, 1, - funaddr, Pmode); + emit_library_call (chkr_check_exec_libfunc, LCT_CONST_MAKE_BLOCK, + VOIDmode, 1, funaddr, Pmode); } emit_queue (); } @@ -1664,7 +1690,7 @@ rtx_for_function_call (fndecl, exp) /* Do the register loads required for any wholly-register parms or any parms which are passed both on the stack and in a register. Their - expressions were already evaluated. + expressions were already evaluated. Mark all register-parms as living through the call, putting these USE insns in the CALL_INSN_FUNCTION_USAGE field. */ @@ -1816,7 +1842,7 @@ try_to_integrate (fndecl, actparms, target, ignore, type, structure_value_addr) outgoing argument list in addition to the requested space, but there is no way to ask for stack space such that an argument list of a certain length can be - safely constructed. + safely constructed. Add the stack space reserved for register arguments, if any, in the inline function. What is really needed is the @@ -1885,7 +1911,7 @@ combine_pending_stack_adjustment_and_call (unadjusted_args_size, just pushed the arguments without adjust the stack here. */ HOST_WIDE_INT unadjusted_alignment; - unadjusted_alignment + unadjusted_alignment = ((stack_pointer_delta + unadjusted_args_size) % preferred_unit_stack_boundary); @@ -1896,27 +1922,127 @@ combine_pending_stack_adjustment_and_call (unadjusted_args_size, -UNADJUSTED_ALIGNMENT modulo the PREFERRED_UNIT_STACK_BOUNDARY. */ /* Begin by trying to pop all the bytes. */ - unadjusted_alignment - = (unadjusted_alignment + unadjusted_alignment + = (unadjusted_alignment - (pending_stack_adjust % preferred_unit_stack_boundary)); adjustment = pending_stack_adjust; /* Push enough additional bytes that the stack will be aligned after the arguments are pushed. */ - if (unadjusted_alignment >= 0) - adjustment -= preferred_unit_stack_boundary - unadjusted_alignment; - else - adjustment += unadjusted_alignment; - + if (preferred_unit_stack_boundary > 1) + { + if (unadjusted_alignment > 0) + adjustment -= preferred_unit_stack_boundary - unadjusted_alignment; + else + adjustment += unadjusted_alignment; + } + /* Now, sets ARGS_SIZE->CONSTANT so that we pop the right number of bytes after the call. The right number is the entire PENDING_STACK_ADJUST less our ADJUSTMENT plus the amount required by the arguments in the first place. */ - args_size->constant + args_size->constant = pending_stack_adjust - adjustment + unadjusted_args_size; return adjustment; } +/* Scan X expression if it does not dereference any argument slots + we already clobbered by tail call arguments (as noted in stored_args_map + bitmap). + Return non-zero if X expression dereferences such argument slots, + zero otherwise. */ + +static int +check_sibcall_argument_overlap_1 (x) + rtx x; +{ + RTX_CODE code; + int i, j; + unsigned int k; + const char *fmt; + + if (x == NULL_RTX) + return 0; + + code = GET_CODE (x); + + if (code == MEM) + { + if (XEXP (x, 0) == current_function_internal_arg_pointer) + i = 0; + else if (GET_CODE (XEXP (x, 0)) == PLUS + && XEXP (XEXP (x, 0), 0) == + current_function_internal_arg_pointer + && GET_CODE (XEXP (XEXP (x, 0), 1)) == CONST_INT) + i = INTVAL (XEXP (XEXP (x, 0), 1)); + else + return 0; + +#ifdef ARGS_GROW_DOWNWARD + i = -i - GET_MODE_SIZE (GET_MODE (x)); +#endif + + for (k = 0; k < GET_MODE_SIZE (GET_MODE (x)); k++) + if (i + k < stored_args_map->n_bits + && TEST_BIT (stored_args_map, i + k)) + return 1; + + return 0; + } + + /* Scan all subexpressions. */ + fmt = GET_RTX_FORMAT (code); + for (i = 0; i < GET_RTX_LENGTH (code); i++, fmt++) + { + if (*fmt == 'e') + { + if (check_sibcall_argument_overlap_1 (XEXP (x, i))) + return 1; + } + else if (*fmt == 'E') + { + for (j = 0; j < XVECLEN (x, i); j++) + if (check_sibcall_argument_overlap_1 (XVECEXP (x, i, j))) + return 1; + } + } + return 0; +} + +/* Scan sequence after INSN if it does not dereference any argument slots + we already clobbered by tail call arguments (as noted in stored_args_map + bitmap). Add stack slots for ARG to stored_args_map bitmap afterwards. + Return non-zero if sequence after INSN dereferences such argument slots, + zero otherwise. */ + +static int +check_sibcall_argument_overlap (insn, arg) + rtx insn; + struct arg_data *arg; +{ + int low, high; + + if (insn == NULL_RTX) + insn = get_insns (); + else + insn = NEXT_INSN (insn); + + for (; insn; insn = NEXT_INSN (insn)) + if (INSN_P (insn) + && check_sibcall_argument_overlap_1 (PATTERN (insn))) + break; + +#ifdef ARGS_GROW_DOWNWARD + low = -arg->offset.constant - arg->size.constant; +#else + low = arg->offset.constant; +#endif + + for (high = low + arg->size.constant; low < high; low++) + SET_BIT (stored_args_map, low); + return insn != NULL_RTX; +} + /* Generate all the code for a function call and return an rtx for its value. Store the value in TARGET (specified as an rtx) if convenient. @@ -1947,7 +2073,6 @@ expand_call (exp, target, ignore) /* Declaration of the function being called, or 0 if the function is computed (not known by name). */ tree fndecl = 0; - char *name = 0; rtx insn; int try_tail_call = 1; int try_tail_recursion = 1; @@ -1993,7 +2118,7 @@ expand_call (exp, target, ignore) int reg_parm_seen; /* Nonzero if this is an indirect function call. */ - /* Nonzero if we must avoid push-insns in the args for this call. + /* Nonzero if we must avoid push-insns in the args for this call. If stack space is allocated for register parameters, but not by the caller, then it is preallocated in the fixed part of the stack frame. So the entire argument block must then be preallocated (i.e., we @@ -2028,7 +2153,7 @@ expand_call (exp, target, ignore) int old_inhibit_defer_pop = inhibit_defer_pop; int old_stack_allocated; rtx call_fusage; - register tree p; + register tree p = TREE_OPERAND (exp, 0); register int i; /* The alignment of the stack, in bits. */ HOST_WIDE_INT preferred_stack_boundary; @@ -2077,12 +2202,17 @@ expand_call (exp, target, ignore) flags |= flags_from_decl_or_type (fndecl); } - /* If we don't have specific function to call, see if we have a + /* If we don't have specific function to call, see if we have a attributes set in the type. */ else + flags |= flags_from_decl_or_type (TREE_TYPE (TREE_TYPE (p))); + + /* Mark if the function returns with the stack pointer depressed. */ + if (TREE_CODE (TREE_TYPE (TREE_TYPE (p))) == FUNCTION_TYPE + && TYPE_RETURNS_STACK_DEPRESSED (TREE_TYPE (TREE_TYPE (p)))) { - p = TREE_OPERAND (exp, 0); - flags |= flags_from_decl_or_type (TREE_TYPE (TREE_TYPE (p))); + flags |= ECF_SP_DEPRESSED; + flags &= ~(ECF_PURE | ECF_CONST); } #ifdef REG_PARM_STACK_SPACE @@ -2132,24 +2262,13 @@ expand_call (exp, target, ignore) structure_value_addr = XEXP (target, 0); else { - /* Assign a temporary to hold the value. */ - tree d; - /* For variable-sized objects, we must be called with a target specified. If we were to allocate space on the stack here, we would have no way of knowing when to free it. */ + rtx d = assign_temp (TREE_TYPE (exp), 1, 1, 1); - if (struct_value_size < 0) - abort (); - - /* This DECL is just something to feed to mark_addressable; - it doesn't get pushed. */ - d = build_decl (VAR_DECL, NULL_TREE, TREE_TYPE (exp)); - DECL_RTL (d) = assign_temp (TREE_TYPE (exp), 1, 0, 1); - mark_addressable (d); - mark_temp_addr_taken (DECL_RTL (d)); - structure_value_addr = XEXP (DECL_RTL (d), 0); - TREE_USED (d) = 1; + mark_temp_addr_taken (d); + structure_value_addr = XEXP (d, 0); target = 0; } } @@ -2167,15 +2286,8 @@ expand_call (exp, target, ignore) return temp; } - if (fndecl && DECL_NAME (fndecl)) - name = IDENTIFIER_POINTER (DECL_NAME (fndecl)); - /* Figure out the amount to which the stack should be aligned. */ -#ifdef PREFERRED_STACK_BOUNDARY preferred_stack_boundary = PREFERRED_STACK_BOUNDARY; -#else - preferred_stack_boundary = STACK_BOUNDARY; -#endif /* Operand 0 is a pointer-to-function; get the type of the function. */ funtype = TREE_TYPE (TREE_OPERAND (exp, 0)); @@ -2255,14 +2367,12 @@ expand_call (exp, target, ignore) or not. */ INIT_CUMULATIVE_ARGS (args_so_far, funtype, NULL_RTX, (fndecl == 0)); - /* Make a vector to hold all the information about each arg. */ - args = (struct arg_data *) alloca (num_actuals - * sizeof (struct arg_data)); - bzero ((char *) args, num_actuals * sizeof (struct arg_data)); + args = (struct arg_data *) alloca (num_actuals * sizeof (struct arg_data)); + memset ((char *) args, 0, num_actuals * sizeof (struct arg_data)); - /* Build up entries inthe ARGS array, compute the size of the arguments - into ARGS_SIZE, etc. */ + /* Build up entries in the ARGS array, compute the size of the + arguments into ARGS_SIZE, etc. */ initialize_argument_information (num_actuals, args, &args_size, n_named_args, actparms, fndecl, &args_so_far, reg_parm_stack_space, @@ -2299,11 +2409,10 @@ expand_call (exp, target, ignore) /* Tail calls can make things harder to debug, and we're traditionally pushed these optimizations into -O2. Don't try if we're already - expanding a call, as that means we're an argument. Similarly, if - there's pending loops or cleanups we know there's code to follow - the call. + expanding a call, as that means we're an argument. Don't try if + there's cleanups, as we know there's code to follow the call. - If rtx_equal_function_value_matters is false, that means we've + If rtx_equal_function_value_matters is false, that means we've finished with regular parsing. Which means that some of the machinery we use to generate tail-calls is no longer in place. This is most often true of sjlj-exceptions, which we couldn't @@ -2312,7 +2421,6 @@ expand_call (exp, target, ignore) if (currently_expanding_call++ != 0 || !flag_optimize_sibling_calls || !rtx_equal_function_value_matters - || !stmt_loop_nest_empty () || any_pending_cleanups (1) || args_size.var) try_tail_call = try_tail_recursion = 0; @@ -2345,6 +2453,7 @@ expand_call (exp, target, ignore) before the sibcall_epilogue. */ || fndecl == NULL_TREE || (flags & (ECF_RETURNS_TWICE | ECF_LONGJMP)) + || TREE_THIS_VOLATILE (fndecl) || !FUNCTION_OK_FOR_SIBCALL (fndecl) /* If this function requires more stack slots than the current function, we cannot change it into a sibling call. */ @@ -2368,7 +2477,7 @@ expand_call (exp, target, ignore) (See unsafe_for_reeval commentary for details.) Generate a new argument list. Pass safe arguments through - unchanged. For the easy badness wrap them in UNSAVE_EXPRs. + unchanged. For the easy badness wrap them in UNSAVE_EXPRs. For hard badness, evaluate them now and put their resulting rtx in a temporary VAR_DECL. @@ -2383,7 +2492,7 @@ expand_call (exp, target, ignore) end = num_actuals; } else - { + { inc = -1; i = num_actuals - 1; end = -1; @@ -2393,26 +2502,27 @@ expand_call (exp, target, ignore) { switch (unsafe_for_reeval (args[i].tree_value)) { - case 0: /* Safe. */ - break; + case 0: /* Safe. */ + break; - case 1: /* Mildly unsafe. */ - args[i].tree_value = unsave_expr (args[i].tree_value); - break; + case 1: /* Mildly unsafe. */ + args[i].tree_value = unsave_expr (args[i].tree_value); + break; - case 2: /* Wildly unsafe. */ - { - tree var = build_decl (VAR_DECL, NULL_TREE, + case 2: /* Wildly unsafe. */ + { + tree var = build_decl (VAR_DECL, NULL_TREE, TREE_TYPE (args[i].tree_value)); - DECL_RTL (var) = expand_expr (args[i].tree_value, NULL_RTX, - VOIDmode, EXPAND_NORMAL); + SET_DECL_RTL (var, + expand_expr (args[i].tree_value, NULL_RTX, + VOIDmode, EXPAND_NORMAL)); args[i].tree_value = var; - } - break; + } + break; - default: - abort (); - } + default: + abort (); + } /* We need to build actparms for optimize_tail_recursion. We can safely trash away TREE_PURPOSE, since it is unused by this function. */ @@ -2436,6 +2546,10 @@ expand_call (exp, target, ignore) int save_pending_stack_adjust = pending_stack_adjust; int save_stack_pointer_delta = stack_pointer_delta; + /* Emit any queued insns now; otherwise they would end up in + only one of the alternates. */ + emit_queue (); + /* Use a new sequence to hold any RTL we generate. We do not even know if we will use this RTL yet. The final decision can not be made until after RTL generation for the entire function is @@ -2472,7 +2586,8 @@ expand_call (exp, target, ignore) is subject to race conditions, just as with multithreaded programs. */ - emit_library_call (gen_rtx_SYMBOL_REF (Pmode, "__bb_fork_func"), 0, + emit_library_call (gen_rtx_SYMBOL_REF (Pmode, "__bb_fork_func"), + LCT_ALWAYS_RETURN, VOIDmode, 0); } @@ -2526,7 +2641,7 @@ expand_call (exp, target, ignore) argblock = 0; call_fusage = 0; - /* Start a new sequence for the normal call case. + /* Start a new sequence for the normal call case. From this point on, if the sibling call fails, we want to set sibcall_failure instead of continuing the loop. */ @@ -2541,11 +2656,6 @@ expand_call (exp, target, ignore) expand_start_target_temps (); } - /* When calling a const function, we must pop the stack args right away, - so that the pop is deleted or moved with the call. */ - if (flags & (ECF_CONST | ECF_PURE)) - NO_DEFER_POP; - /* Don't let pending stack adjusts add up to too much. Also, do all pending adjustments now if there is any chance this might be a call to alloca or if we are expanding a sibling @@ -2555,11 +2665,15 @@ expand_call (exp, target, ignore) || pass == 0) do_pending_stack_adjust (); + /* When calling a const function, we must pop the stack args right away, + so that the pop is deleted or moved with the call. */ + if (flags & (ECF_CONST | ECF_PURE)) + NO_DEFER_POP; + /* Push the temporary stack slot level so that we can free any temporaries we make. */ push_temp_slots (); - #ifdef FINAL_REG_PARM_STACK_SPACE reg_parm_stack_space = FINAL_REG_PARM_STACK_SPACE (args_size.constant, args_size.var); @@ -2580,16 +2694,21 @@ expand_call (exp, target, ignore) pattern, do not round up, since we'll be re-using whatever space our caller provided. */ unadjusted_args_size - = compute_argument_block_size (reg_parm_stack_space, &adjusted_args_size, + = compute_argument_block_size (reg_parm_stack_space, + &adjusted_args_size, (pass == 0 ? 0 : preferred_stack_boundary)); - old_stack_allocated = stack_pointer_delta - pending_stack_adjust; + old_stack_allocated = stack_pointer_delta - pending_stack_adjust; /* The argument block when performing a sibling call is the incoming argument block. */ if (pass == 0) - argblock = virtual_incoming_args_rtx; + { + argblock = virtual_incoming_args_rtx; + stored_args_map = sbitmap_alloc (args_size.constant); + sbitmap_zero (stored_args_map); + } /* If we have no actual push instructions, or shouldn't use them, make space for all args right now. */ @@ -2637,7 +2756,7 @@ expand_call (exp, target, ignore) Therefore, we save any area of the stack that was already written and that we are using. Here we set up to do this by making a new stack usage map from the old one. The - actual save will be done by store_one_arg. + actual save will be done by store_one_arg. Another approach might be to try to reorder the argument evaluations to avoid this conflicting stack usage. */ @@ -2660,11 +2779,11 @@ expand_call (exp, target, ignore) = (char *) alloca (highest_outgoing_arg_in_use); if (initial_highest_arg_in_use) - bcopy (initial_stack_usage_map, stack_usage_map, - initial_highest_arg_in_use); + memcpy (stack_usage_map, initial_stack_usage_map, + initial_highest_arg_in_use); if (initial_highest_arg_in_use != highest_outgoing_arg_in_use) - bzero (&stack_usage_map[initial_highest_arg_in_use], + memset (&stack_usage_map[initial_highest_arg_in_use], 0, (highest_outgoing_arg_in_use - initial_highest_arg_in_use)); needed = 0; @@ -2672,10 +2791,10 @@ expand_call (exp, target, ignore) /* The address of the outgoing argument list must not be copied to a register here, because argblock would be left pointing to the wrong place after the call to - allocate_dynamic_stack_space below. */ + allocate_dynamic_stack_space below. */ argblock = virtual_outgoing_args_rtx; - } + } else { if (inhibit_defer_pop == 0) @@ -2683,7 +2802,7 @@ expand_call (exp, target, ignore) /* Try to reuse some or all of the pending_stack_adjust to get this space. */ needed - = (combine_pending_stack_adjustment_and_call + = (combine_pending_stack_adjustment_and_call (unadjusted_args_size, &adjusted_args_size, preferred_unit_stack_boundary)); @@ -2703,7 +2822,7 @@ expand_call (exp, target, ignore) do_pending_stack_adjust (); needed = 0; } - else + else /* We need to allocate space. We'll do that in push_block below. */ pending_stack_adjust = 0; @@ -2753,7 +2872,7 @@ expand_call (exp, target, ignore) /* Make a new map for the new argument list. */ stack_usage_map = (char *) alloca (highest_outgoing_arg_in_use); - bzero (stack_usage_map, highest_outgoing_arg_in_use); + memset (stack_usage_map, 0, highest_outgoing_arg_in_use); highest_outgoing_arg_in_use = 0; } allocate_dynamic_stack_space (push_size, NULL_RTX, @@ -2773,7 +2892,6 @@ expand_call (exp, target, ignore) compute_argument_addresses (args, argblock, num_actuals); -#ifdef PREFERRED_STACK_BOUNDARY /* If we push args individually in reverse order, perform stack alignment before the first push (the last arg). */ if (PUSH_ARGS_REVERSED && argblock == 0 @@ -2781,12 +2899,12 @@ expand_call (exp, target, ignore) { /* When the stack adjustment is pending, we get better code by combining the adjustments. */ - if (pending_stack_adjust + if (pending_stack_adjust && ! (flags & (ECF_CONST | ECF_PURE)) && ! inhibit_defer_pop) { pending_stack_adjust - = (combine_pending_stack_adjustment_and_call + = (combine_pending_stack_adjustment_and_call (unadjusted_args_size, &adjusted_args_size, preferred_unit_stack_boundary)); @@ -2799,12 +2917,6 @@ expand_call (exp, target, ignore) /* Now that the stack is properly aligned, pops can't safely be deferred during the evaluation of the arguments. */ NO_DEFER_POP; -#endif - - /* Don't try to defer pops if preallocating, not even from the first arg, - since ARGBLOCK probably refers to the SP. */ - if (argblock) - NO_DEFER_POP; funexp = rtx_for_function_call (fndecl, exp); @@ -2840,8 +2952,17 @@ expand_call (exp, target, ignore) for (i = 0; i < num_actuals; i++) if (args[i].reg == 0 || args[i].pass_on_stack) - store_one_arg (&args[i], argblock, flags, - adjusted_args_size.var != 0, reg_parm_stack_space); + { + rtx before_arg = get_last_insn (); + + if (store_one_arg (&args[i], argblock, flags, + adjusted_args_size.var != 0, + reg_parm_stack_space) + || (pass == 0 + && check_sibcall_argument_overlap (before_arg, + &args[i]))) + sibcall_failure = 1; + } /* If we have a parm that is passed in registers but not in memory and whose alignment does not permit a direct copy into registers, @@ -2855,23 +2976,30 @@ expand_call (exp, target, ignore) if (reg_parm_seen) for (i = 0; i < num_actuals; i++) if (args[i].partial != 0 && ! args[i].pass_on_stack) - store_one_arg (&args[i], argblock, flags, - adjusted_args_size.var != 0, reg_parm_stack_space); + { + rtx before_arg = get_last_insn (); + + if (store_one_arg (&args[i], argblock, flags, + adjusted_args_size.var != 0, + reg_parm_stack_space) + || (pass == 0 + && check_sibcall_argument_overlap (before_arg, + &args[i]))) + sibcall_failure = 1; + } -#ifdef PREFERRED_STACK_BOUNDARY /* If we pushed args in forward order, perform stack alignment after pushing the last arg. */ if (!PUSH_ARGS_REVERSED && argblock == 0) anti_adjust_stack (GEN_INT (adjusted_args_size.constant - unadjusted_args_size)); -#endif /* If register arguments require space on the stack and stack space was not preallocated, allocate stack space here for arguments passed in registers. */ #ifdef OUTGOING_REG_PARM_STACK_SPACE if (!ACCUMULATE_OUTGOING_ARGS - && must_preallocate == 0 && reg_parm_stack_space > 0) + && must_preallocate == 0 && reg_parm_stack_space > 0) anti_adjust_stack (GEN_INT (reg_parm_stack_space)); #endif @@ -2886,9 +3014,9 @@ expand_call (exp, target, ignore) /* Mark the memory for the aggregate as write-only. */ if (current_function_check_memory_usage) - emit_library_call (chkr_set_right_libfunc, 1, + emit_library_call (chkr_set_right_libfunc, LCT_CONST_MAKE_BLOCK, VOIDmode, 3, - structure_value_addr, ptr_mode, + structure_value_addr, ptr_mode, GEN_INT (struct_value_size), TYPE_MODE (sizetype), GEN_INT (MEMORY_USE_WO), @@ -2899,10 +3027,10 @@ expand_call (exp, target, ignore) } funexp = prepare_call_address (funexp, fndecl, &call_fusage, - reg_parm_seen); + reg_parm_seen, pass == 0); load_register_parameters (args, num_actuals, &call_fusage, flags); - + /* Perform postincrements before actually calling the function. */ emit_queue (); @@ -2924,11 +3052,9 @@ expand_call (exp, target, ignore) /* All arguments and registers used for the call must be set up by now! */ -#ifdef PREFERRED_STACK_BOUNDARY /* Stack must be properly aligned now. */ if (pass && stack_pointer_delta % preferred_unit_stack_boundary) abort (); -#endif /* Generate the actual call instruction. */ emit_call_1 (funexp, fndecl, funtype, unadjusted_args_size, @@ -2938,8 +3064,8 @@ expand_call (exp, target, ignore) /* Verify that we've deallocated all the stack we used. */ if (pass - && old_stack_allocated != stack_pointer_delta - pending_stack_adjust) - abort(); + && old_stack_allocated != stack_pointer_delta - pending_stack_adjust) + abort (); /* If call is cse'able, make appropriate pair of reg-notes around it. Test valreg so we don't crash; may safely ignore `const' @@ -2973,7 +3099,7 @@ expand_call (exp, target, ignore) gen_rtx_SCRATCH (VOIDmode))), note); emit_libcall_block (insns, temp, valreg, note); - + valreg = temp; } else if (flags & (ECF_CONST | ECF_PURE)) @@ -2989,7 +3115,7 @@ expand_call (exp, target, ignore) rtx temp = gen_reg_rtx (GET_MODE (valreg)); rtx last, insns; - /* The return value from a malloc-like function is a pointer. */ + /* The return value from a malloc-like function is a pointer. */ if (TREE_CODE (TREE_TYPE (exp)) == POINTER_TYPE) mark_reg_pointer (temp, BIGGEST_ALIGNMENT); @@ -2998,7 +3124,7 @@ expand_call (exp, target, ignore) /* The return value from a malloc-like function can not alias anything else. */ last = get_last_insn (); - REG_NOTES (last) = + REG_NOTES (last) = gen_rtx_EXPR_LIST (REG_NOALIAS, temp, REG_NOTES (last)); /* Write out the sequence. */ @@ -3012,9 +3138,9 @@ expand_call (exp, target, ignore) if nonvolatile values are live. For functions that cannot return, inform flow that control does not fall through. */ - if ((flags & (ECF_RETURNS_TWICE | ECF_NORETURN | ECF_LONGJMP)) || pass == 0) + if ((flags & (ECF_NORETURN | ECF_LONGJMP)) || pass == 0) { - /* The barrier or NOTE_INSN_SETJMP note must be emitted + /* The barrier must be emitted immediately after the CALL_INSN. Some ports emit more than just a CALL_INSN above, so we must search for it here. */ @@ -3027,13 +3153,7 @@ expand_call (exp, target, ignore) abort (); } - if (flags & ECF_RETURNS_TWICE) - { - emit_note_after (NOTE_INSN_SETJMP, last); - current_function_calls_setjmp = 1; - } - else - emit_barrier_after (last); + emit_barrier_after (last); } if (flags & ECF_LONGJMP) @@ -3088,18 +3208,20 @@ expand_call (exp, target, ignore) The Irix 6 ABI has examples of this. */ else if (GET_CODE (valreg) == PARALLEL) { - int bytes = int_size_in_bytes (TREE_TYPE (exp)); - if (target == 0) { - target = assign_stack_temp (TYPE_MODE (TREE_TYPE (exp)), - bytes, 0); - MEM_SET_IN_STRUCT_P (target, AGGREGATE_TYPE_P (TREE_TYPE (exp))); + /* This will only be assigned once, so it can be readonly. */ + tree nt = build_qualified_type (TREE_TYPE (exp), + (TYPE_QUALS (TREE_TYPE (exp)) + | TYPE_QUAL_CONST)); + + target = assign_temp (nt, 0, 1, 1); preserve_temp_slots (target); } if (! rtx_equal_p (target, valreg)) - emit_group_store (target, valreg, bytes, + emit_group_store (target, valreg, + int_size_in_bytes (TREE_TYPE (exp)), TYPE_ALIGN (TREE_TYPE (exp))); /* We can not support sibling calls for this case. */ @@ -3118,7 +3240,12 @@ expand_call (exp, target, ignore) emit_move_insn (target, valreg); } else if (TYPE_MODE (TREE_TYPE (exp)) == BLKmode) - target = copy_blkmode_from_reg (target, valreg, TREE_TYPE (exp)); + { + target = copy_blkmode_from_reg (target, valreg, TREE_TYPE (exp)); + + /* We can not support sibling calls for this case. */ + sibcall_failure = 1; + } else target = copy_to_reg (valreg); @@ -3131,13 +3258,25 @@ expand_call (exp, target, ignore) { tree type = TREE_TYPE (exp); int unsignedp = TREE_UNSIGNED (type); + int offset = 0; /* If we don't promote as expected, something is wrong. */ if (GET_MODE (target) != promote_mode (type, TYPE_MODE (type), &unsignedp, 1)) abort (); - target = gen_rtx_SUBREG (TYPE_MODE (type), target, 0); + if ((WORDS_BIG_ENDIAN || BYTES_BIG_ENDIAN) + && GET_MODE_SIZE (GET_MODE (target)) + > GET_MODE_SIZE (TYPE_MODE (type))) + { + offset = GET_MODE_SIZE (GET_MODE (target)) + - GET_MODE_SIZE (TYPE_MODE (type)); + if (! BYTES_BIG_ENDIAN) + offset = (offset / UNITS_PER_WORD) * UNITS_PER_WORD; + else if (! WORDS_BIG_ENDIAN) + offset %= UNITS_PER_WORD; + } + target = gen_rtx_SUBREG (TYPE_MODE (type), target, offset); SUBREG_PROMOTED_VAR_P (target) = 1; SUBREG_PROMOTED_UNSIGNED_P (target) = unsignedp; } @@ -3146,7 +3285,7 @@ expand_call (exp, target, ignore) /* If size of args is variable or this was a constructor call for a stack argument, restore saved stack-pointer value. */ - if (old_stack_level) + if (old_stack_level && ! (flags & ECF_SP_DEPRESSED)) { emit_stack_restore (SAVE_BLOCK, old_stack_level, NULL_RTX); pending_stack_adjust = old_pending_adj; @@ -3188,7 +3327,7 @@ expand_call (exp, target, ignore) stack_usage_map = initial_stack_usage_map; } - /* If this was alloca, record the new stack level for nonlocal gotos. + /* If this was alloca, record the new stack level for nonlocal gotos. Check for the handler slots since we might not have a save area for non-local gotos. */ @@ -3217,10 +3356,6 @@ expand_call (exp, target, ignore) { tail_call_insns = insns; - /* If something prevents making this a sibling call, - zero out the sequence. */ - if (sibcall_failure) - tail_call_insns = NULL_RTX; /* Restore the pending stack adjustment now that we have finished generating the sibling call sequence. */ @@ -3228,15 +3363,22 @@ expand_call (exp, target, ignore) stack_pointer_delta = save_stack_pointer_delta; /* Prepare arg structure for next iteration. */ - for (i = 0 ; i < num_actuals ; i++) + for (i = 0; i < num_actuals; i++) { args[i].value = 0; args[i].aligned_regs = 0; args[i].stack = 0; } + + sbitmap_free (stored_args_map); } else normal_call_insns = insns; + + /* If something prevents making this a sibling call, + zero out the sequence. */ + if (sibcall_failure) + tail_call_insns = NULL_RTX; } /* The function optimize_sibling_and_tail_recursive_calls doesn't @@ -3245,7 +3387,7 @@ expand_call (exp, target, ignore) function who's expansion contains another CALL_PLACEHOLDER. If there are any C_Ps in any of these sequences, replace them - with their normal call. */ + with their normal call. */ for (insn = normal_call_insns; insn; insn = NEXT_INSN (insn)) if (GET_CODE (insn) == CALL_INSN @@ -3285,34 +3427,29 @@ expand_call (exp, target, ignore) currently_expanding_call--; - return target; -} - -/* Returns nonzero if FUN is the symbol for a library function which can - not throw. */ - -static int -libfunc_nothrow (fun) - rtx fun; -{ - if (fun == throw_libfunc - || fun == rethrow_libfunc - || fun == sjthrow_libfunc - || fun == sjpopnthrow_libfunc) - return 0; + /* If this function returns with the stack pointer depressed, ensure + this block saves and restores the stack pointer, show it was + changed, and adjust for any outgoing arg space. */ + if (flags & ECF_SP_DEPRESSED) + { + clear_pending_stack_adjust (); + emit_insn (gen_rtx (CLOBBER, VOIDmode, stack_pointer_rtx)); + emit_move_insn (virtual_stack_dynamic_rtx, stack_pointer_rtx); + save_stack_pointer (); + } - return 1; + return target; } /* Output a library call to function FUN (a SYMBOL_REF rtx). - The RETVAL parameter specifies whether return value needs to be saved, other - parameters are documented in the emit_library_call function bellow. */ + The RETVAL parameter specifies whether return value needs to be saved, other + parameters are documented in the emit_library_call function below. */ static rtx emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) int retval; rtx orgfun; rtx value; - int fn_type; + enum libcall_type fn_type; enum machine_mode outmode; int nargs; va_list p; @@ -3328,8 +3465,16 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) struct args_size alignment_pad; rtx argblock = 0; CUMULATIVE_ARGS args_so_far; - struct arg { rtx value; enum machine_mode mode; rtx reg; int partial; - struct args_size offset; struct args_size size; rtx save_area; }; + struct arg + { + rtx value; + enum machine_mode mode; + rtx reg; + int partial; + struct args_size offset; + struct args_size size; + rtx save_area; + }; struct arg *argvec; int old_inhibit_defer_pop = inhibit_defer_pop; rtx call_fusage = 0; @@ -3337,15 +3482,16 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) rtx valreg; int pcc_struct_value = 0; int struct_value_size = 0; - int flags = 0; + int flags; int reg_parm_stack_space = 0; int needed; + rtx before_call; #ifdef REG_PARM_STACK_SPACE /* Define the boundary of the register parm stack space that needs to be save, if any. */ int low_to_save = -1, high_to_save = 0; - rtx save_area = 0; /* Place that it is saved */ + rtx save_area = 0; /* Place that it is saved. */ #endif /* Size of the stack reserved for parameter registers. */ @@ -3360,21 +3506,38 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) #endif #endif - if (fn_type == 1) - flags |= ECF_CONST; - else if (fn_type == 2) - flags |= ECF_PURE; - fun = orgfun; + /* By default, library functions can not throw. */ + flags = ECF_NOTHROW; - if (libfunc_nothrow (fun)) - flags |= ECF_NOTHROW; + switch (fn_type) + { + case LCT_NORMAL: + case LCT_CONST: + case LCT_PURE: + /* Nothing to do here. */ + break; + case LCT_CONST_MAKE_BLOCK: + flags |= ECF_CONST; + break; + case LCT_PURE_MAKE_BLOCK: + flags |= ECF_PURE; + break; + case LCT_NORETURN: + flags |= ECF_NORETURN; + break; + case LCT_THROW: + flags = ECF_NORETURN; + break; + case LCT_ALWAYS_RETURN: + flags = ECF_ALWAYS_RETURN; + break; + } + fun = orgfun; -#ifdef PREFERRED_STACK_BOUNDARY /* Ensure current function's preferred stack boundary is at least what we need. */ if (cfun->preferred_stack_boundary < PREFERRED_STACK_BOUNDARY) cfun->preferred_stack_boundary = PREFERRED_STACK_BOUNDARY; -#endif /* If this kind of value comes back in memory, decide where in memory it should come back. */ @@ -3393,7 +3556,7 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) if (value != 0 && GET_CODE (value) == MEM) mem_value = value; else - mem_value = assign_stack_temp (outmode, GET_MODE_SIZE (outmode), 0); + mem_value = assign_temp (type_for_mode (outmode, 0), 0, 1, 1); #endif /* This call returns a big structure. */ @@ -3410,9 +3573,13 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) library functions shouldn't have many args. */ argvec = (struct arg *) alloca ((nargs + 1) * sizeof (struct arg)); - bzero ((char *) argvec, (nargs + 1) * sizeof (struct arg)); + memset ((char *) argvec, 0, (nargs + 1) * sizeof (struct arg)); +#ifdef INIT_CUMULATIVE_LIBCALL_ARGS + INIT_CUMULATIVE_LIBCALL_ARGS (args_so_far, outmode, fun); +#else INIT_CUMULATIVE_ARGS (args_so_far, NULL_TREE, fun, 0); +#endif args_size.constant = 0; args_size.var = 0; @@ -3457,7 +3624,6 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) NULL_TREE, &args_size, &argvec[count].offset, &argvec[count].size, &alignment_pad); - if (argvec[count].reg == 0 || argvec[count].partial != 0 || reg_parm_stack_space > 0) args_size.constant += argvec[count].size.constant; @@ -3496,12 +3662,44 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) #ifdef FUNCTION_ARG_PASS_BY_REFERENCE if (FUNCTION_ARG_PASS_BY_REFERENCE (args_so_far, mode, NULL_TREE, 1)) { - /* We do not support FUNCTION_ARG_CALLEE_COPIES here since it can - be viewed as just an efficiency improvement. */ - rtx slot = assign_stack_temp (mode, GET_MODE_SIZE (mode), 0); - emit_move_insn (slot, val); - val = force_operand (XEXP (slot, 0), NULL_RTX); + rtx slot; + int must_copy = 1 +#ifdef FUNCTION_ARG_CALLEE_COPIES + && ! FUNCTION_ARG_CALLEE_COPIES (args_so_far, mode, + NULL_TREE, 1) +#endif + ; + + if (GET_MODE (val) == MEM && ! must_copy) + slot = val; + else if (must_copy) + { + slot = assign_temp (type_for_mode (mode, 0), 0, 1, 1); + emit_move_insn (slot, val); + } + else + { + tree type = type_for_mode (mode, 0); + + slot = gen_rtx_MEM (mode, + expand_expr (build1 (ADDR_EXPR, + build_pointer_type + (type), + make_tree (type, val)), + NULL_RTX, VOIDmode, 0)); + } + + call_fusage = gen_rtx_EXPR_LIST (VOIDmode, + gen_rtx_USE (VOIDmode, slot), + call_fusage); + if (must_copy) + call_fusage = gen_rtx_EXPR_LIST (VOIDmode, + gen_rtx_CLOBBER (VOIDmode, + slot), + call_fusage); + mode = Pmode; + val = force_operand (XEXP (slot, 0), NULL_RTX); } #endif @@ -3519,7 +3717,7 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) locate_and_pad_parm (mode, NULL_TREE, #ifdef STACK_PARMS_IN_REG_PARM_AREA - 1, + 1, #else argvec[count].reg != 0, #endif @@ -3548,14 +3746,12 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) assemble_external_libcall (fun); original_args_size = args_size; -#ifdef PREFERRED_STACK_BOUNDARY args_size.constant = (((args_size.constant + stack_pointer_delta + STACK_BYTES - 1) / STACK_BYTES * STACK_BYTES) - stack_pointer_delta); -#endif args_size.constant = MAX (args_size.constant, reg_parm_stack_space); @@ -3600,20 +3796,23 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) stack_usage_map = (char *) alloca (highest_outgoing_arg_in_use); if (initial_highest_arg_in_use) - bcopy (initial_stack_usage_map, stack_usage_map, - initial_highest_arg_in_use); + memcpy (stack_usage_map, initial_stack_usage_map, + initial_highest_arg_in_use); if (initial_highest_arg_in_use != highest_outgoing_arg_in_use) - bzero (&stack_usage_map[initial_highest_arg_in_use], + memset (&stack_usage_map[initial_highest_arg_in_use], 0, highest_outgoing_arg_in_use - initial_highest_arg_in_use); needed = 0; - /* The address of the outgoing argument list must not be copied to a - register here, because argblock would be left pointing to the - wrong place after the call to allocate_dynamic_stack_space below. - */ + /* We must be careful to use virtual regs before they're instantiated, + and real regs afterwards. Loop optimization, for example, can create + new libcalls after we've instantiated the virtual regs, and if we + use virtuals anyway, they won't match the rtl patterns. */ - argblock = virtual_outgoing_args_rtx; + if (virtuals_instantiated) + argblock = plus_constant (stack_pointer_rtx, STACK_POINTER_OFFSET); + else + argblock = virtual_outgoing_args_rtx; } else { @@ -3621,13 +3820,11 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) argblock = push_block (GEN_INT (args_size.constant), 0, 0); } -#ifdef PREFERRED_STACK_BOUNDARY /* If we push args individually in reverse order, perform stack alignment before the first push (the last arg). */ if (argblock == 0 && PUSH_ARGS_REVERSED) anti_adjust_stack (GEN_INT (args_size.constant - original_args_size.constant)); -#endif if (PUSH_ARGS_REVERSED) { @@ -3655,7 +3852,7 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) for (count = 0; count < reg_parm_stack_space; count++) #endif { - if (count >= highest_outgoing_arg_in_use + if (count >= highest_outgoing_arg_in_use || stack_usage_map[count] == 0) continue; @@ -3681,7 +3878,7 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) stack_area = gen_rtx_MEM (save_mode, memory_address (save_mode, plus_constant (argblock, - - high_to_save))); + -high_to_save))); #else stack_area = gen_rtx_MEM (save_mode, memory_address (save_mode, @@ -3702,7 +3899,7 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) } } #endif - + /* Push the args that need to be pushed. */ /* ARGNUM indexes the ARGVEC array in the order in which the arguments @@ -3742,7 +3939,7 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) if (i != upper_bound) { /* We need to make a save area. See what mode we can make - it. */ + it. */ enum machine_mode save_mode = mode_for_size (argvec[argnum].size.constant * BITS_PER_UNIT, @@ -3773,20 +3970,18 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) } } -#ifdef PREFERRED_STACK_BOUNDARY /* If we pushed args in forward order, perform stack alignment after pushing the last arg. */ if (argblock == 0 && !PUSH_ARGS_REVERSED) anti_adjust_stack (GEN_INT (args_size.constant - original_args_size.constant)); -#endif if (PUSH_ARGS_REVERSED) argnum = nargs - 1; else argnum = 0; - fun = prepare_call_address (fun, NULL_TREE, &call_fusage, 0); + fun = prepare_call_address (fun, NULL_TREE, &call_fusage, 0, 0); /* Now load any reg parms into their regs. */ @@ -3828,7 +4023,7 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) force_operand (XEXP (mem_value, 0), NULL_RTX))); if (GET_CODE (struct_value_rtx) == REG) - use_reg (&call_fusage, struct_value_rtx); + use_reg (&call_fusage, struct_value_rtx); } /* Don't allow popping to be deferred, since then @@ -3837,11 +4032,11 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) valreg = (mem_value == 0 && outmode != VOIDmode ? hard_libcall_value (outmode) : NULL_RTX); -#ifdef PREFERRED_STACK_BOUNDARY /* Stack must be properly aligned now. */ if (stack_pointer_delta & (PREFERRED_STACK_BOUNDARY / BITS_PER_UNIT - 1)) - abort(); -#endif + abort (); + + before_call = get_last_insn (); /* We pass the old value of inhibit_defer_pop + 1 to emit_call_1, which will set inhibit_defer_pop to that value. */ @@ -3850,16 +4045,38 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) always signed. We also assume that the list of arguments passed has no impact, so we pretend it is unknown. */ - emit_call_1 (fun, - get_identifier (XSTR (orgfun, 0)), + emit_call_1 (fun, + get_identifier (XSTR (orgfun, 0)), build_function_type (outmode == VOIDmode ? void_type_node : type_for_mode (outmode, 0), NULL_TREE), - original_args_size.constant, args_size.constant, + original_args_size.constant, args_size.constant, struct_value_size, FUNCTION_ARG (args_so_far, VOIDmode, void_type_node, 1), valreg, old_inhibit_defer_pop + 1, call_fusage, flags); + /* For calls to `setjmp', etc., inform flow.c it should complain + if nonvolatile values are live. For functions that cannot return, + inform flow that control does not fall through. */ + + if (flags & (ECF_NORETURN | ECF_LONGJMP)) + { + /* The barrier note must be emitted + immediately after the CALL_INSN. Some ports emit more than + just a CALL_INSN above, so we must search for it here. */ + + rtx last = get_last_insn (); + while (GET_CODE (last) != CALL_INSN) + { + last = PREV_INSN (last); + /* There was no CALL_INSN? */ + if (last == before_call) + abort (); + } + + emit_barrier_after (last); + } + /* Now restore inhibit_defer_pop to its actual original value. */ OK_DEFER_POP; @@ -3946,7 +4163,7 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) PARM_BOUNDARY); } #endif - + /* If we saved any argument areas, restore them. */ for (count = 0; count < nargs; count++) if (argvec[count].save_area) @@ -3983,8 +4200,8 @@ emit_library_call_value_1 (retval, orgfun, value, fn_type, outmode, nargs, p) (use (memory (scratch)). */ void -emit_library_call VPARAMS((rtx orgfun, int fn_type, enum machine_mode outmode, - int nargs, ...)) +emit_library_call VPARAMS((rtx orgfun, enum libcall_type fn_type, + enum machine_mode outmode, int nargs, ...)) { #ifndef ANSI_PROTOTYPES rtx orgfun; @@ -4017,7 +4234,8 @@ emit_library_call VPARAMS((rtx orgfun, int fn_type, enum machine_mode outmode, If VALUE is nonzero, VALUE is returned. */ rtx -emit_library_call_value VPARAMS((rtx orgfun, rtx value, int fn_type, +emit_library_call_value VPARAMS((rtx orgfun, rtx value, + enum libcall_type fn_type, enum machine_mode outmode, int nargs, ...)) { #ifndef ANSI_PROTOTYPES @@ -4092,17 +4310,19 @@ target_for_arg (type, size, args_addr, offset) or 0 on a machine where arguments are pushed individually. MAY_BE_ALLOCA nonzero says this could be a call to `alloca' - so must be careful about how the stack is used. + so must be careful about how the stack is used. VARIABLE_SIZE nonzero says that this was a variable-sized outgoing argument stack. This is used if ACCUMULATE_OUTGOING_ARGS to indicate that we need not worry about saving and restoring the stack. - FNDECL is the declaration of the function we are calling. */ + FNDECL is the declaration of the function we are calling. -static void -store_one_arg (arg, argblock, flags, variable_size, - reg_parm_stack_space) + Return non-zero if this arg should cause sibcall failure, + zero otherwise. */ + +static int +store_one_arg (arg, argblock, flags, variable_size, reg_parm_stack_space) struct arg_data *arg; rtx argblock; int flags; @@ -4114,9 +4334,10 @@ store_one_arg (arg, argblock, flags, variable_size, int partial = 0; int used = 0; int i, lower_bound = 0, upper_bound = 0; + int sibcall_failure = 0; if (TREE_CODE (pval) == ERROR_MARK) - return; + return 1; /* Push a new temporary level for any temporaries we make for this argument. */ @@ -4165,15 +4386,15 @@ store_one_arg (arg, argblock, flags, variable_size, if (save_mode == BLKmode) { - arg->save_area = assign_stack_temp (BLKmode, - arg->size.constant, 0); - MEM_SET_IN_STRUCT_P (arg->save_area, - AGGREGATE_TYPE_P (TREE_TYPE - (arg->tree_value))); + tree ot = TREE_TYPE (arg->tree_value); + tree nt = build_qualified_type (ot, (TYPE_QUALS (ot) + | TYPE_QUAL_CONST)); + + arg->save_area = assign_temp (nt, 0, 1, 1); preserve_temp_slots (arg->save_area); emit_block_move (validize_mem (arg->save_area), stack_area, - GEN_INT (arg->size.constant), - PARM_BOUNDARY); + expr_size (arg->tree_value), + MIN (PARM_BOUNDARY, TYPE_ALIGN (nt))); } else { @@ -4205,7 +4426,7 @@ store_one_arg (arg, argblock, flags, variable_size, here. */ if (arg->n_aligned_regs != 0) reg = 0; - + /* If this is being passed partially in a register, we can't evaluate it directly into its stack slot. Otherwise, we can. */ if (arg->value == 0) @@ -4258,8 +4479,8 @@ store_one_arg (arg, argblock, flags, variable_size, /* If the value is already in the stack slot, we are done. */ if (current_function_check_memory_usage && GET_CODE (arg->stack) == MEM) { - emit_library_call (chkr_set_right_libfunc, 1, VOIDmode, 3, - XEXP (arg->stack, 0), Pmode, + emit_library_call (chkr_set_right_libfunc, LCT_CONST_MAKE_BLOCK, + VOIDmode, 3, XEXP (arg->stack, 0), Pmode, ARGS_SIZE_RTX (arg->size), TYPE_MODE (sizetype), GEN_INT (MEMORY_USE_RW), @@ -4273,7 +4494,7 @@ store_one_arg (arg, argblock, flags, variable_size, /* Argument is a scalar, not entirely passed in registers. (If part is passed in registers, arg->partial says how much and emit_push_insn will take care of putting it there.) - + Push it, and if its size is less than the amount of space allocated to it, also bump stack pointer by the additional space. @@ -4331,6 +4552,63 @@ store_one_arg (arg, argblock, flags, variable_size, size_rtx = expr_size (pval); } + if ((flags & ECF_SIBCALL) && GET_CODE (arg->value) == MEM) + { + /* emit_push_insn might not work properly if arg->value and + argblock + arg->offset areas overlap. */ + rtx x = arg->value; + int i = 0; + + if (XEXP (x, 0) == current_function_internal_arg_pointer + || (GET_CODE (XEXP (x, 0)) == PLUS + && XEXP (XEXP (x, 0), 0) == + current_function_internal_arg_pointer + && GET_CODE (XEXP (XEXP (x, 0), 1)) == CONST_INT)) + { + if (XEXP (x, 0) != current_function_internal_arg_pointer) + i = INTVAL (XEXP (XEXP (x, 0), 1)); + + /* expand_call should ensure this */ + if (arg->offset.var || GET_CODE (size_rtx) != CONST_INT) + abort (); + + if (arg->offset.constant > i) + { + if (arg->offset.constant < i + INTVAL (size_rtx)) + sibcall_failure = 1; + } + else if (arg->offset.constant < i) + { + if (i < arg->offset.constant + INTVAL (size_rtx)) + sibcall_failure = 1; + } + } + } + + /* If parm is passed both in stack and in register and offset is + greater than reg_parm_stack_space, split the offset. */ + if (arg->reg && arg->pass_on_stack) + { + if (arg->offset.constant < reg_parm_stack_space && arg->offset.var) + error ("variable offset is passed paritially in stack and in reg"); + else if (arg->offset.constant < reg_parm_stack_space && arg->size.var) + error ("variable size is passed partially in stack and in reg"); + else if (arg->offset.constant < reg_parm_stack_space + && ((arg->offset.constant + arg->size.constant) + > reg_parm_stack_space)) + { + rtx size_rtx1 = GEN_INT (reg_parm_stack_space - arg->offset.constant); + emit_push_insn (arg->value, arg->mode, TREE_TYPE (pval), size_rtx1, + TYPE_ALIGN (TREE_TYPE (pval)), partial, reg, + excess, argblock, ARGS_SIZE_RTX (arg->offset), + reg_parm_stack_space, + ARGS_SIZE_RTX (arg->alignment_pad)); + + size_rtx = GEN_INT (INTVAL(size_rtx) - reg_parm_stack_space); + } + } + + emit_push_insn (arg->value, arg->mode, TREE_TYPE (pval), size_rtx, TYPE_ALIGN (TREE_TYPE (pval)), partial, reg, excess, argblock, ARGS_SIZE_RTX (arg->offset), @@ -4338,9 +4616,8 @@ store_one_arg (arg, argblock, flags, variable_size, ARGS_SIZE_RTX (arg->alignment_pad)); } - /* Unless this is a partially-in-register argument, the argument is now - in the stack. + in the stack. ??? Note that this can change arg->value from arg->stack to arg->stack_slot and it matters when they are not the same. @@ -4362,4 +4639,6 @@ store_one_arg (arg, argblock, flags, variable_size, preserve_temp_slots (NULL_RTX); free_temp_slots (); pop_temp_slots (); + + return sibcall_failure; }