This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: move alpha move expanders
- To: Matt Kraai <kraai at alumni dot carnegiemellon dot edu>
- Subject: Re: move alpha move expanders
- From: Richard Henderson <rth at redhat dot com>
- Date: Wed, 5 Sep 2001 11:37:48 -0700
- Cc: gcc-patches at gcc dot gnu dot org
- References: <20010905095843.A3018@opensource.lineo.com>
On Wed, Sep 05, 2001 at 09:58:43AM -0600, Matt Kraai wrote:
> I have two questions about alpha_expand_mov. In the following if
> statement, it seems like the HOST_BITS_PER_WIDE_INT >= 64 test is
> redundant, since i1 is set to -(i0 < 0) in this case:
>
> if (HOST_BITS_PER_WIDE_INT >= 64 || i1 == -(i0 < 0))
The HOST_BITS_PER_WIDE_INT test is a compile-time constant.
Redundant tests like this let us avoid the run-time test
that follows.
> Second, the next if statement tests !temp, but I don't see why
> temp must be initialized at this point:
Sure enough, it should be initialized to null. Didn't see
this because of the compile-time test above evaluating true
in a native build.
r~