[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-changelog] [xen stable-4.1] x86: explicit suffix in inline assembler (for clang).
commit e737f237a373c673537062a7a3324f345120a113 Author: Tim Deegan <tim@xxxxxxx> AuthorDate: Fri Aug 16 12:11:21 2013 +0200 Commit: Jan Beulich <jbeulich@xxxxxxxx> CommitDate: Fri Aug 16 12:11:21 2013 +0200 x86: explicit suffix in inline assembler (for clang). This fixes the clang build, and has no effect on gcc's output. Signed-off-by: Tim Deegan <tim@xxxxxxx> Committed-by: Jan Beulich <jbeulich@xxxxxxxx> master commit: 59a28b5f045331641cbf0c1fc8d5d67afe328939 master date: 2013-02-14 14:20:06 +0100 Note that this isn't just a build fix - if the "delta" input ends up in memory, gas would default to 32-bit operand size (and should really warn about the ambiguity). --- xen/arch/x86/time.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/xen/arch/x86/time.c b/xen/arch/x86/time.c index 20edfd7..c3b0b86 100644 --- a/xen/arch/x86/time.c +++ b/xen/arch/x86/time.c @@ -140,7 +140,7 @@ static inline u64 scale_delta(u64 delta, struct time_scale *scale) : "a" ((u32)delta), "1" ((u32)(delta >> 32)), "2" (scale->mul_frac) ); #else asm ( - "mul %2 ; shrd $32,%1,%0" + "mulq %2 ; shrd $32,%1,%0" : "=a" (product), "=d" (delta) : "rm" (delta), "0" ((u64)scale->mul_frac) ); #endif -- generated by git-patchbot for /home/xen/git/xen.git#stable-4.1 _______________________________________________ Xen-changelog mailing list Xen-changelog@xxxxxxxxxxxxx http://lists.xensource.com/xen-changelog
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |