bug#30832: Please add support for new architecture "riscv64" (RISC-V 64 bits little-endian)

bug#30832: Please add support for new architecture "riscv64" (RISC-V 64 bits little-endian)

[Manuel A. Fernandez Montecelo]

[-- Attachment #1: Type: text/plain, Size: 828 bytes --]

Hello,

Please add support for this architecture, simple patch attached (based
on guile-2.0, the most popular version in Debian).

The change is simply to add "riscv64" in the following location, the
arch name seems to be missing in:

  http://git.savannah.gnu.org/cgit/guile.git/tree/module/system/base/target.scm#n74

Since it ends in -64, the pointer size should be detected correctly in
the current code.

I compiled this and ran the test suite successfully:

Totals for this test run:
passes:                 40317
failures:               0
unexpected passes:      0
expected failures:      9
unresolved test cases:  571
untested test cases:    1
unsupported test cases: 10
errors:                 0

If there's anything missing, please let me know.


Cheers.
-- 
Manuel A. Fernandez Montecelo <manuel.montezelo@gmail.com>

[-- Attachment #2: riscv64.patch --]
[-- Type: text/x-patch, Size: 754 bytes --]

Index: guile-2.0-2.0.13+1/module/system/base/target.scm
===================================================================
--- guile-2.0-2.0.13+1.orig/module/system/base/target.scm
+++ guile-2.0-2.0.13+1/module/system/base/target.scm
@@ -65,7 +65,7 @@
       (cond ((string-match "^i[0-9]86$" cpu)
              (endianness little))
             ((member cpu '("x86_64" "ia64"
-                           "powerpcle" "powerpc64le" "mipsel" "mips64el" "nios2" "sh4" "alpha"))
+                           "powerpcle" "powerpc64le" "mipsel" "mips64el" "nios2" "sh4" "alpha" "riscv64"))
              (endianness little))
             ((member cpu '("sparc" "sparc64" "powerpc" "powerpc64" "spu"
                            "mips" "mips64" "m68k" "s390x"))

bug#30832: Please add support for new architecture "riscv64" (RISC-V 64 bits little-endian)

[Mark H Weaver]

"Manuel A. Fernandez Montecelo" <manuel.montezelo@gmail.com> writes:
> Please add support for this architecture, simple patch attached (based
> on guile-2.0, the most popular version in Debian).

This should be fixed now, using a patch provided earlier by Shea Levy
which also covers riscv32.  See commit
08bb4628369cf3efe7c96a32919705c41a9ed9e9 on the stable-2.2 branch and
d6e669b8cb26f870ea6611a54788b75fbad67bbe on the stable-2.0 branch.

I'm closing this bug now, but please reopen if there are still issues.

     Thanks!
       Mark

bug#30832: Please add support for new architecture "riscv64" (RISC-V 64 bits little-endian)

[Manuel A. Fernandez Montecelo]

2018-03-16 5:21 GMT+01:00 Mark H Weaver <mhw@netris.org>:
> "Manuel A. Fernandez Montecelo" <manuel.montezelo@gmail.com> writes:
>> Please add support for this architecture, simple patch attached (based
>> on guile-2.0, the most popular version in Debian).
>
> This should be fixed now, using a patch provided earlier by Shea Levy
> which also covers riscv32.  See commit
> 08bb4628369cf3efe7c96a32919705c41a9ed9e9 on the stable-2.2 branch and
> d6e669b8cb26f870ea6611a54788b75fbad67bbe on the stable-2.0 branch.
>
> I'm closing this bug now, but please reopen if there are still issues.

Ah, it's fine then.  I assumed that addition of new features started
in master and then some changes flowed down to stable branches when
needed.

Thanks!

-- 
Manuel A. Fernandez Montecelo <manuel.montezelo@gmail.com>