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

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

Yametazamwa mara 10
Ruka hadi kwenye ujumbe wa kwanza ambao haujasomwa

Steve Peters

hayajasomwa,
12 Apr 2007, 12:53:2512/04/2007
kwa 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

hayajasomwa,
13 Apr 2007, 03:02:3113/04/2007
kwa 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

Ujumbe 0 mpya