rmuir commented on PR #14901: URL: https://github.com/apache/lucene/pull/14901#issuecomment-3045577281
> We should only apply this PR on main branch, because I don't want people to get sudden slowdowns in 10.x branch. Who would get a slowdown? If they are impacted by this PR, they've probably got a 10x slowdown happening somewhere already :) I think the ongoing risk would be, that developer adds a new function to PanamaVectorUtilSupport.java, backports it, and then it causes a new 10x-20x slowdown. That's the motivation behind this PR anyway. It shouldn't slow anyone down, just requires AVX2 on amd64 machines before we try to vectorize. IMO best to disable vectorization completely in such cases, it will actually prevent a 10x slowdown. "supporting" some SSSE-based 2x speedup for a couple floating point methods isn't worth it? -- 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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
