Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss

[perl #42475] [PATCH] Change asm() to __asm__()

8 views
Skip to first unread message

Steve Peters

unread,
Apr 12, 2007, 12:53:25 PM4/12/07
to bugs-bi...@rt.perl.org
# New Ticket Created by Steve Peters
# Please include the string: [perl #42475]
# in the subject line of all future correspondence about this issue.
# <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=42475 >


-ansi -pedantic choke on the call to asm() in src/ops/core.ops. This is
because asm() is gcc-builtin. However, if you use __asm_() instead, everything
works just fine. The following patch gets Parrot to compile just file,
although rather loudly with -ansi -pedantic.

Steve Peters
st...@fisharerojo.org


Index: src/ops/core.ops
===================================================================
--- src/ops/core.ops (revision 18166)
+++ src/ops/core.ops (working copy)
@@ -107,7 +107,7 @@
inline op cpu_ret() {
#ifdef __GNUC__
# ifdef I386
- asm("ret");
+ __asm__("ret");
# endif
#endif
goto NEXT();

Chromatic

unread,
Apr 13, 2007, 3:02:31 AM4/13/07
to perl6-i...@perl.org, Steve Peters, bugs-bi...@rt.perl.org
On Thursday 12 April 2007 09:53, Steve Peters wrote:

> -ansi -pedantic choke on the call to asm() in src/ops/core.ops. This is
> because asm() is gcc-builtin. However, if you use __asm_() instead,
> everything works just fine. The following patch gets Parrot to compile
> just file, although rather loudly with -ansi -pedantic.

Thanks, applied as r18176.

-- c

0 new messages