On Tue, 23 Jun 2026 12:17:30 GMT, Ferenc Rakoczi <[email protected]> wrote:
>> Ferenc Rakoczi has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Unite x86 and aarch64 for UseIntPolyIntrinsics for AOTCache. > > I think it is rather unfortunate that this method was added to this > microbenchmark suite as its contribution to the run time of any real crypto > operation is minimal, so it makes almost no difference if it runs twice as > fast. However, it is important that it runs in constant time (i.e. its > running time is independent of the values in its input arrays and, more > importantly, whether the value of the "set" argument is 0 or 1). The java > code was written in such a way, but there is no guarantee that the compiler > will not change it back to using a branch instead of the xors if it can > figure out that only those 2 values are possible for "set". So the intrinsic > here is more for guaranteeing "set" value independent execution than for any > performance gains. @ferakocz Thanks for your patience! ------------- PR Comment: https://git.openjdk.org/jdk/pull/30941#issuecomment-4788240642
