RFR: 8288047: Accelerate Poly1305 on x86_64 using AVX512 instructions [v14]

Volodymyr Paprotski duke at openjdk.org
Fri Nov 11 20:10:33 UTC 2022


On Fri, 11 Nov 2022 19:56:40 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

>> I believe its needed.
>> 
>> TLDR.. Couple of check ins ago, I broke the 32-bit build, and that was the 'easy' fix..
>
> Right, `addq` instructions are x64-specific. I was confused because `assembler_x86.hpp` doesn't declare them as such which is a bug.

I am mystified at how it actually gets removed from the `assembler_x86.o` object on 32-bit.. The only reliable/portable way _would_ be with `#ifdef` but its not there.. so.. code-generation? `sed`-like preprocessing? Can one edit object files after the gcc ran? The build must be doing something clever!! Haven't seen it yet.. 

Whatever the trick is, `assembler_x86.hpp` gets it, but not `macroAssembler_x86.hpp`.

If it doesn't ring any bells, maybe I will spend some more time looking at the traces, maybe can figure out what the build script is doing to remove the symbol.

-------------

PR: https://git.openjdk.org/jdk/pull/10582


More information about the security-dev mailing list