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: gcc intrinsics for am33 parallel integer arithmetic


On Sep  1, 2000, Matthew Hiller <hiller@cygnus.com> wrote:

> +   if (!lvalue_p_func)
> +     warning ("could not find a function for testing the validity of "
> + 	     "operands 1 and 3 of dsp intrinsic");

String concatenation isn't portable.  Besides, I'm not sure I like the
idea of having a run-time error for this kind of thing.  I'd rather
require each front-end to define lvalue_p ().  It can be done
trivially for chill, by simply calling chill_location() (or renaming
it), and it shouldn't be too hard for any other languages to define
it.

But then, I'm not entitled to impose such a requirement on all
front-ends, as a maintainer of mn10300, so I'd appreciate a second
opinion :-)


Other than that and the suggestions from Graham Stott, I think the
patch is in very good shape.

-- 
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


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