[Last Call] Learn about multicloud storage options and how to improve your company's cloud strategy. Register Now

x
?
Solved

Assembly compile error

Posted on 2004-08-19
14
Medium Priority
?
552 Views
Last Modified: 2008-02-01
HI,

I am trying to compile a package called TMPI.  I am facing a problem with this part of the code. I am compiling this for a machine that has a athlon processor and using gcc as a compiler


tmpi_i386.h
----

#ifndef _I386_H_
#define _I386_H_

#ifdef __SMP__
#define LOCK_PREFIX "lock ; "
#else
#define LOCK_PREFIX ""
#endif

/* some hacks to defeat gcc over-optimizations */
struct __dummy { unsigned long a[100]; };
#define ADDR (*(volatile struct __dummy *)addr)

#endif

----------------

# include "tmpi_i386.h"

__inline__ int cas32(volatile int *addr, int old, int new)
{
    char ret;

    __asm__ __volatile__(LOCK_PREFIX
                         "cmpxchg %2, %1\n\t"   /* swap values if current value
                                                   is known */
                         "sete %0"              /* read success/fail from EF */
                         : "=q" (ret), "=m" (ADDR)
                         : "r" (new), "a" (old)
                                                 : "%eax" );
    return (int)ret;
}

int cas64(volatile long long *ptr, long long old, long long new);

__inline__ int casp(void * *pval, void * old, void * new)
{
        /**
         * somehow I have to decide the size of (void *)
         * during the runtime.
         */
        if (sizeof(void *)==sizeof(int))
                return cas32((int*)pval, (int)old, (int)new);
        else
                return cas64((long long *)pval, (long long)old, (long long)new);
}

__inline__ int test_and_set(volatile void *addr, int bit)
{
    int oldbit;

    __asm__ __volatile__(LOCK_PREFIX
                         "btsl %2, %1\n\t"  /* set bit, store old in CF */
                         "sbbl %0, %0"      /* oldbit -= CF */
                         : "=r" (oldbit),"=m" (ADDR)
                         : "Ir" (bit));

    return (-oldbit);
}

__inline__ int test_and_clear(volatile void *addr, int bit)
{
    int oldbit;

    __asm__ __volatile__(LOCK_PREFIX
                         "btrl %2, %1\n\t"  /* clear bit, store old in CF */
                         "sbbl %0, %0"      /* oldbit -= CF */
                         : "=r" (oldbit), "=m" (ADDR)
                         : "Ir" (bit));

    return (-oldbit);
}

__inline__ int test_and_change(volatile void *addr, int bit)
{
    int oldbit;

 __asm__ __volatile__(LOCK_PREFIX
                         "btcl %2, %1\n\t"  /* change bit, store in CF */
                         "sbbl %0, %0"      /* oldbit -= CF */
                         : "=r" (oldbit), "=m" (ADDR)
                         : "Ir" (bit));

    return (-oldbit);
}

__inline__ int anf32(volatile int *ptr, int val)
{
        int old;

        if (!val) return *ptr;

        do {
                old=*ptr;
        } while (!cas32(ptr, old, old+val));

        return old+val;
}

__inline__ int snf32(volatile int *ptr, int val)
{
        int old;

        if (!val) return *ptr;

        do {
                old=*ptr;
        } while (!cas32(ptr, old, old-val));

        return old-val;
}
__inline__ int fna32(volatile int *ptr, int val)
{
        int old;

        if (!val) return *ptr;

        do {
                old=*ptr;
        } while (!cas32(ptr, old, old+val));

        return old;
}

__inline__ int fns32(volatile int *ptr, int val)
{
        int old;

        if (!val) return *ptr;

        do {
                old=*ptr;
        } while (!cas32(ptr, old, old-val));

        return old;
}

__inline__ long long anf64(volatile long long *ptr, long long val)
{
        long long old;

        if (!val) return *ptr;

        do {
                old=*ptr;
        } while (!cas64(ptr, old, old+val));

        return old+val;
}

__inline__ int snf64(volatile long long *ptr, long long val)
{
        long long old;

        if (!val) return *ptr;

        do {
                old=*ptr;
        } while (!cas64(ptr, old, old-val));

        return old-val;
}

---------------------------

error :

cc -O -Wall -I. -D__SMP__   -c -o atomic.o atomic.c
atomic.c: In function `casp':
atomic.c:28: warning: cast from pointer to integer of different size
atomic.c:28: warning: cast from pointer to integer of different size
atomic.c: In function `cas32':
atomic.c:13: Invalid `asm' statement:
atomic.c:13: fixed or forbidden register 0 (ax) was spilled for class AREG.
atomic.c: In function `casp':
atomic.c:13: Invalid `asm' statement:
atomic.c:13: fixed or forbidden register 0 (ax) was spilled for class AREG.
atomic.c: In function `anf32':
atomic.c:13: Invalid `asm' statement:
atomic.c:13: fixed or forbidden register 0 (ax) was spilled for class AREG.
atomic.c: In function `snf32':
atomic.c:13: Invalid `asm' statement:
atomic.c:13: fixed or forbidden register 0 (ax) was spilled for class AREG.
atomic.c: In function `fna32':
atomic.c:13: Invalid `asm' statement:
atomic.c:13: fixed or forbidden register 0 (ax) was spilled for class AREG.
atomic.c: In function `fns32':
atomic.c:13: Invalid `asm' statement:
atomic.c:13: fixed or forbidden register 0 (ax) was spilled for class AREG.
make: *** [atomic.o] Error 1
-------------------------------------------
0
Comment
Question by:team
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
14 Comments
 
LVL 45

Expert Comment

by:sunnycoder
ID: 11839282
The code is meant for i386 machines ... since you are trying to compile it on athlon it will not work ... you can either use a cross compiler to compile it (I am not sure if you have a cross assembler to generate athlon exe from i386 assembly but will search) or you can rewrite it to equivalent athlon assembly code
0
 
LVL 45

Expert Comment

by:sunnycoder
ID: 11839333
0
 

Author Comment

by:team
ID: 11839352
err i thought athlon instruction set was based on x86 and gcc shouldn't be really complaining.
0
Industry Leaders: We Want Your Opinion!

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 

Author Comment

by:team
ID: 11839359
i will check out tdasm
0
 
LVL 45

Expert Comment

by:sunnycoder
ID: 11839373
I am not an assembly expert but I really do not think both of them have same assembly.

Wait for someone more experienced with assembly to comment on this .... It might also help to post a pointer to this question in the assembly section
http://www.experts-exchange.com/Programming/Programming_Languages/Assembly/
0
 

Author Comment

by:team
ID: 11839394
error compiling tdasm: wonder if i should start a seperate thread on this.

 ***************************************************************************/
#include <stdio.h>
#include <string>
#include <vector>
#include "tablecomp.h"
struct {
        int lx;
        const char *ls;
} lex2str[] = {             <-------------------------------
#ifndef BISON_Y_TAB_H
# define BISON_Y_TAB_H

#ifndef YYSTYPE
# define YYSTYPE yystype
# define YYSTYPE_IS_TRIVIAL 1
#endif
# define        ENDIAN  257
# define        ENUM    258
# define        INSTFORMAT      259
# define        OPERAND 260
-----------------------



sed -f printyylex.sed <tablecomp.h >printyylex.cc
c++ -DHAVE_CONFIG_H -I. -I. -I..     -Wall -pedantic -ansi -g3 -O0  -c printyylex.cc
printyylex.cc:102: parse error at end of input
printyylex.cc:24: storage size of `lex2str' isn't known
make[2]: *** [printyylex.o] Error 1
0
 
LVL 45

Expert Comment

by:sunnycoder
ID: 11839415
you ran configure before make ?
0
 
LVL 45

Expert Comment

by:sunnycoder
ID: 11839418
looks like tdasm has not been updated for quite sometime ... May be we should be looking for something more recent
0
 
LVL 22

Expert Comment

by:grg99
ID: 11840188
Athlon is 99.9999% code compatible with x86, so that's not the problem.

The problem is the code is trying to use some very fancy __asm tricks.

The "register spilled" problem is because the compiler thinks the __asm code is trying to use ax in a conflicting way-- as an operand and as a result.  You might be able to get around it by turning down the optimization level or using some other gcc option that tells gcc to not think so hard over this sequence.

One roundabout way:
I would suggest for starters you think bout using some system API to do these test-and-set operations.  Most os-s have API's for this.  They're a tad slower than an inline lock-and-special-CPU-instruction, but a lot less finicky.



0
 

Author Comment

by:team
ID: 11842936
well it turned out to be a gcc bug in prior versions of gcc-3.0 . I had used 2.95 earlier and now used 3.2.2 and the errors got reduced to

cc -O -Wall -I. -D__SMP__   -c -o atomic.o atomic.c
atomic.c: In function `casp':
atomic.c:28: warning: cast from pointer to integer of different size
atomic.c:28: warning: cast from pointer to integer of different size
atomic.c: In function `cas32':
atomic.c:7: can't find a register in class `AREG' while reloading `asm'
make[2]: *** [atomic.o] Error 1
make[1]: *** [machine] Error 2
make: *** [src] Error 2
0
 

Author Comment

by:team
ID: 11842954
sunnycoder: yes i did run configure
0
 
LVL 49

Expert Comment

by:DanRollins
ID: 11865596
I recommend PAQ and refund:  grg99's suggestion is a good one (even if it is not the specific solution to this particular problem) and the team's finding of bugs in old versions of gcc is worth retaining in the database.

-- Dan
0
 

Accepted Solution

by:
CetusMOD earned 0 total points
ID: 11916980
Closed, 500 points refunded.
CetusMOD
Community Support Moderator
0

Featured Post

VIDEO: THE CONCERTO CLOUD FOR HEALTHCARE

Modern healthcare requires a modern cloud. View this brief video to understand how the Concerto Cloud for Healthcare can help your organization.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Have you thought about creating an iPhone application (app), but didn't even know where to get started? Here's how: ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ ~ Important pre-programming comments: I’ve never tri…
This is a short and sweet, but (hopefully) to the point article. There seems to be some fundamental misunderstanding about the function prototype for the "main" function in C and C++, more specifically what type this function should return. I see so…
The goal of this video is to provide viewers with basic examples to understand how to use strings and some functions related to them in the C programming language.
The goal of this video is to provide viewers with basic examples to understand how to create, access, and change arrays in the C programming language.

656 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question