Fix, was: Re: [Fwd: CVS trunk bootstrap failure on alphaev6-unknown-linux-gnu]

Alexandre Oliva aoliva@redhat.com
Mon Jul 9 08:02:00 GMT 2001


On Jul  8, 2001, Toon Moene <toon@moene.indiv.nluug.nl> wrote:

> OK to install ?

I think it would still be a good idea to pass the mode as an argument,
just in case the two other arguments are CONST_INTs (remember that
CONST_INTs don't have a mode).  In fact, your proposed patch would
probably crash or just fail to do right thing should the first
argument be a CONST_INT.

However, since all uses of these instructions take a register as an
argument (and only now, after having written the paragraph above, do I
see it can only make sense to call have_add2_insn having a REG in the
first argument, I think it may indeed be ok.  Then perhaps it would be
enough to check that GET_MODE (x) != VOIDmode, and abort () otherwise.

-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                  aoliva@{cygnus.com, redhat.com}
CS PhD student at IC-Unicamp        oliva@{lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist    *Please* write to mailing lists, not to me



More information about the Gcc-bugs mailing list