[GitHub] [lucene] jpountz commented on pull request #69: LUCENE-9850: Use PFOR encoding for doc IDs (instead of FOR)

[GitBox <gi...@apache.org>]

jpountz commented on pull request #69:
URL: https://github.com/apache/lucene/pull/69#issuecomment-815845326


   > I think we leave the implementation as is and hope that we can do something better with more explicit vectorization support in the future
   
   (Sorry replying here as Github prevents me from replying on the existing thread)
   
   +1 Let go with whichever of `arr[i] = IDENTITY_PLUS_ONE[i] * val + base` or `arr[i] = (i+1) * val + base` runs fastest in your micro benchmark. We can still improve things later if we find a way to trick the JVM into auto-vectorizing this loop.


-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org