This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: gcj bytecode generation patch


Anthony Green <green@cygnus.com> writes:

> +	  {
> +	    /* Pointers use 1 stack slot no matter what the target's
> +               pointer size is.  */
> +	    if (TREE_CODE (TREE_TYPE (exp)) == POINTER_TYPE)
> +	      NOTE_POP (1);
> +	    else
> +	      NOTE_POP (TYPE_PRECISION (TREE_TYPE (exp)) > 32 ? 2 : 1);
> +	  }

I would combine the two cases, perhaps like this:

              int is_wide = (TREE_CODE (TREE_TYPE (exp)) != POINTER_TYPE
                             && TYPE_PRECISION (TREE_TYPE (exp)) > 32);
	      NOTE_POP (is_wide ? 2 : 1);
-- 
	--Per Bothner
per@bothner.com   http://www.bothner.com/~per/

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]