[PATCH,SH] 64-bit integer comparison is incorrect

Kaz Kojima kkojima@rr.iij4u.or.jp
Wed Apr 16 07:57:00 GMT 2008


Christian BRUEL <christian.bruel@st.com> wrote:
> here it is.

Although I'm not sure if it's mandatory for testcases, it's
better to follow the coding standard, isn't it?
If so, the comment at the top would be formatted like as

/* Problem only noticed on SH for -mcbranchdi DImode comparison with
   constants.  Target dependant failure but test valid for alls.  */

and spaces are needed between the function symbol and '('.
Also I guess that using abort is the usual way when multiple
tests are done in main:

extern void abort (void);

int main ()
{
  if (test2n (-1LL))
    abort ();

  if (test2n (-2LL))
    abort ();
  ...

Regards,
	kaz



More information about the Gcc-patches mailing list