mccullocht commented on PR #13133: URL: https://github.com/apache/lucene/pull/13133#issuecomment-1964602685
@rmuir @uschindler Would it be sufficient to add a factory function to `VectorizationProvider` for this like `GroupVIntUtil.Decoder createGroupVIntDecoder(MemorySegment segment)`? I anticipated that his might be necessary so I already have a scalar implementation available. I'm surprised by how slow this is with AVX off given that this can be implemented with SSE2 :(. I don't think this can be auto-vectorized by the VM (and FWIW LLVM can't autovectorize it either). Given that the loop would be to a constant bound it is likely something would unroll it anyway and we'd end up with the same code. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
