> Dennis Clarke <dcla...@blastwave.org> writes:
>
>>      for (argno = 0; argno < argc; argno++) {
>>              if (argno < 6)
>>                      *tsp++ = reg[REG_O0 + argno] = va_arg(ap, long);
>>              else
>>                      *tsp++ = va_arg(ap, long);
>
> This is broken.  The arguments are of type int, not long.

To be more compliant with IEEE Std 1003.1-2001 ( ISO/IEC 9899:1999
standard, Section 6.2.5 ) they should be int32_t or similar as
per stdint.h.

But hey, I was just thinking out loud and more concerned about a
functional go compiler in gcc 4.7 RC. I am currently digging around
in snapshots 4.7-20120107.

Dennis

-- 
--
http://pgp.mit.edu:11371/pks/lookup?op=vindex&search=0x1D936C72FA35B44B
+-------------------------+-----------------------------------+
| Dennis Clarke           | Solaris and Linux and Open Source |
| dcla...@blastwave.org   | Respect for open standards.       |
+-------------------------+-----------------------------------+

Reply via email to