travisstaloch / simdjzon

simdjson port to zig
107 stars 5 forks source link

Simplify is_ascii #19

Closed Validark closed 1 year ago

Validark commented 1 year ago

The trick here is performed automatically by the compiler for targets that only support 32-wide vectors. However, on zen4, for example, this trick is not optimal and disables the compiler from generating better code. This change enables the compiler to do this optimization only when necessary.

travisstaloch commented 1 year ago

I didn't know the compiler would do the split and |. Another nice find! :100: