s-yata / marisa-trie

MARISA: Matching Algorithm with Recursively Implemented StorAge
Other
506 stars 89 forks source link

Add wordsize detection for LoongArch64. #56

Open Dandan336 opened 6 months ago

Dandan336 commented 6 months ago

Compiling the marisa failed for loongarch64 in my local system.

$ git clone https://github.com/s-yata/marisa-trie.git
$ cd marisa-trie
$ autoreconf -i
$ ./configure
$ make
$ make check

The results of make check are as follows,

make[3]: Entering directory '/home/zdd/marisa-trie/tests'
FAIL: base-test
PASS: io-test
PASS: vector-test
PASS: trie-test
PASS: marisa-test
============================
Testsuite summary for marisa 0.2.6
============================
# TOTAL: 5
# PASS:  4
# SKIP:  0
# XFAIL: 0
# FAIL:  1
# XPASS: 0
# ERROR: 0

I have added wordsize detection for LoongArch64, please review. I have also submitted a bug request to Debian BTS, the BugID can be found at https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1065467 BTW, the documentations of the LoongArch architecture can be found at the following links, ISA: https://loongson.github.io/LoongArch-Documentation/LoongArch-Vol1-EN.html ABI: https://loongson.github.io/LoongArch-Documentation/LoongArch-ELF-ABI-EN.html

thanks, Dandan Zhang

ncopa commented 4 months ago

There is a better way to solve this: https://github.com/s-yata/marisa-trie/pull/58