diff options
author | Paolo Bonzini <pbonzini@redhat.com> | 2015-05-26 15:15:32 +0200 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2015-05-26 15:15:41 +0200 |
commit | d28c79853ec28eb3ce84675a6a59caba10c39c35 (patch) | |
tree | 3a0446fb422dedc26e6f8460b9d5f38626da15c6 | |
parent | d1da38e16592f6f541d0fa9aa675fa296585f779 (diff) | |
download | qboot-d28c79853ec28eb3ce84675a6a59caba10c39c35.zip qboot-d28c79853ec28eb3ce84675a6a59caba10c39c35.tar.gz qboot-d28c79853ec28eb3ce84675a6a59caba10c39c35.tar.bz2 |
use = for input/output operands
The input side is already handled by the duplicate constraints, and clang
rejects usage of "+".
Reported-by: David Halls (via github)
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
-rw-r--r-- | include/ioport.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/ioport.h b/include/ioport.h index bbec982..208be41 100644 --- a/include/ioport.h +++ b/include/ioport.h @@ -3,12 +3,12 @@ static inline void outsb(unsigned short port, void *buf, int len) { - asm volatile("rep outsb %%ds:(%0), %3" : "+S" (buf), "+c" (len) : "m"(buf), "Nd"(port), "0" (buf), "1" (len)); + asm volatile("rep outsb %%ds:(%0), %3" : "=S" (buf), "=c" (len) : "m"(buf), "Nd"(port), "0" (buf), "1" (len)); } static inline void insb(void *buf, unsigned short port, int len) { - asm volatile("rep insb %3, %%es:(%0)" : "+D" (buf), "+c" (len), "=m"(buf) : "Nd"(port), "0" (buf), "1" (len)); + asm volatile("rep insb %3, %%es:(%0)" : "=D" (buf), "=c" (len), "=m"(buf) : "Nd"(port), "0" (buf), "1" (len)); } static inline unsigned char inb(unsigned short port) |