Messages in this thread | | | From | David Howells <> | Subject | Re: [PATCH 11/23] make atomic_read() and atomic_set() behavior consistent on m32r | Date | Thu, 23 Aug 2007 21:05:40 +0100 |
| |
Segher Boessenkool <segher@kernel.crashing.org> wrote:
> This simply isn't true. The compiler *can* combine asm stuff: > > > typedef struct { int counter; } atomic_t; > > static inline __attribute__((pure)) int atomic_read(const atomic_t *v) > { > int x; > asm("ld %0,@%1" : "=r"(x) : "r"(&v->counter), "m"(v->counter)); > return x; > }
That's not precisely combining asm stuff. The compiler is ditching a whole function because you've told it it can cache the result.
David - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |