[PATCH 1/1] Add "&" asm constraint modifier to ASM_MUL outputs

[PATCH 1/1] Add "&" asm constraint modifier to ASM_MUL outputs

[Rob Browning]

Without this change, building for arm{hf,el} with gcc-4.9 (Debian
4.9.1-14) fails as follows:

  /tmp/ccdZTkXQ.s:14591: rdhi, rdlo and rm must all be different

Thanks to Mark H Weaver <mhw@netris.org> for the suggestion.

---

 This should apply against 2.0.11.

 libguile/vm-i-scheme.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libguile/vm-i-scheme.c b/libguile/vm-i-scheme.c
index 587aa95..35e618b 100644
--- a/libguile/vm-i-scheme.c
+++ b/libguile/vm-i-scheme.c
@@ -363,7 +363,7 @@ VM_DEFINE_FUNCTION (149, ge, "ge?", 2)
       {									\
 	scm_t_signed_bits rlo, rhi;					\
 	asm ("smull %0, %1, %2, %3\n"					\
-	     : "=r" (rlo), "=r" (rhi)					\
+	     : "=&r" (rlo), "=&r" (rhi)					\
 	     : "r" (SCM_UNPACK (x) - scm_tc2_int),			\
 	       "r" (SCM_I_INUM (y)));					\
 	if (SCM_LIKELY (SCM_SRS (rlo, 31) == rhi))			\
-- 
2.1.0

Re: [PATCH 1/1] Add "&" asm constraint modifier to ASM_MUL outputs

[Mark H Weaver]

Rob Browning <rlb@defaultvalue.org> writes:

> Without this change, building for arm{hf,el} with gcc-4.9 (Debian
> 4.9.1-14) fails as follows:
>
>   /tmp/ccdZTkXQ.s:14591: rdhi, rdlo and rm must all be different
>
> Thanks to Mark H Weaver <mhw@netris.org> for the suggestion.

Sorry, I pushed my own equivalent commit before I saw your email,
a85c78ea1393985fdb6e6678dea19135c553d341 on the stable-2.0 branch.

FYI, the commit log above does not conform to our conventions, so that
would have needed fixing.  See the existing logs for examples.

    Thanks!
      Mark