Changeset ba371e1 in mainline

Timestamp:

2010-02-20T22:04:13Z (15 years ago)

Author:

Martin Decky <martin@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

aa30c7f

Parents:

7af8c0e

Message:

conclude Clang support
this is the final piece which was missing to make HelenOS not only compilable with Clang, but make it actually run properly

Location:

kernel/arch

Files:

• amd64/include/atomic.h (modified) (1 diff)
• ia32/include/atomic.h (modified) (1 diff)

kernel/arch/amd64/include/atomic.h

@@ -101 +101 @@
 static inline atomic_count_t test_and_set(atomic_t *val)
 {
-        atomic_count_t v;
+        atomic_count_t v = 1;
         
         asm volatile (
-                "movq $1, %[v]\n"
                 "xchgq %[v], %[count]\n"
-                : [v] "=r" (v),
+                : [v] "+r" (v),
                   [count] "+m" (val->count)
         );

kernel/arch/ia32/include/atomic.h

@@ -101 +101 @@
 static inline atomic_count_t test_and_set(atomic_t *val)
 {
-        atomic_count_t v;
+        atomic_count_t v = 1;
         
         asm volatile (
-                "movl $1, %[v]\n"
                 "xchgl %[v], %[count]\n"
-                : [v] "=r" (v),
+                : [v] "+r" (v),
                   [count] "+m" (val->count)
         );