Skip to content

Conversation

@heiher
Copy link
Contributor

@heiher heiher commented Aug 28, 2025

simd/lines_fwd/134217728
                        time:   [4.3447 ms 4.3474 ms 4.3505 ms]
                        thrpt:  [28.732 GiB/s 28.753 GiB/s 28.771 GiB/s]
                 change:
                        time:   [−1.8221% −1.7185% −1.6176%] (p = 0.00 < 0.05)
                        thrpt:  [+1.6442% +1.7485% +1.8559%]
                        Performance has improved.

```
simd/lines_fwd/134217728
                        time:   [4.3447 ms 4.3474 ms 4.3505 ms]
                        thrpt:  [28.732 GiB/s 28.753 GiB/s 28.771 GiB/s]
                 change:
                        time:   [−1.8221% −1.7185% −1.6176%] (p = 0.00 < 0.05)
                        thrpt:  [+1.6442% +1.7485% +1.8559%]
                        Performance has improved.
```
Copy link
Member

@lhecker lhecker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, that's very interesting! I was reading a lot of RISC-V lately (my first time doing so) and this reminds me a lot of it. I hope LoongArch continuous to gain popularity. 🙂

@lhecker
Copy link
Member

lhecker commented Aug 28, 2025

Apropos, I'm possibly looking to implement this algorithm in the future: http://0x80.pl/notesen/2018-10-18-simd-byte-lookup.html
I could use it for my syntax highlighting PR where I implemented a custom regex engine. Perhaps this could also be of interest for you? I think it may find some great reuse as other regex engines use similar approaches. I'm just not sure if it'll be easily translatable to LoongArch (or even NEON for that matter).

@lhecker lhecker merged commit ccfebb2 into microsoft:main Aug 28, 2025
3 checks passed
@heiher heiher deleted the loong-simd-opt branch August 29, 2025 05:18
@heiher
Copy link
Contributor Author

heiher commented Aug 30, 2025

I'm just not sure if it'll be easily translatable to LoongArch (or even NEON for that matter).

It seems that LoongArch SIMD should be able to handle this without much trouble.

Lou32Verbose pushed a commit to Lou32Verbose/edit that referenced this pull request Jan 11, 2026
jenia90 pushed a commit to jenia90/edit that referenced this pull request Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants