uschindler commented on PR #14901: URL: https://github.com/apache/lucene/pull/14901#issuecomment-3045674157
> There's nothing to test. This change makes it so developers only support avx2 and avx512. That's the idea. > > I will fix the name of the confusing constant and consider inverting the logic to call it something like "ancientIntel". It's not related to anything about integers anymore. If you want vectors on x86 you need AVX256 or AVX512. Please read again what I said: I want the test mode (where we randomize also preferred bitsize) to also always execute the correctness of the integer code. Testing can be slow, but the integer code has to be tested. Therefor please remove the UOE in test mode. That's plain simple. If you don't agree with that let's remove all test-mode code. Now it is incomplete. -- 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
