On Tue, 8 Apr 2025 15:53:05 GMT, Roman Marchenko <rmarche...@openjdk.org> wrote:

>> All the crashes are on "`movaps`" instructions, like "`movaps xmmword ptr 
>> [esi+0x30], xmm0`".
>> 
>> "`movaps`" must operate with aligned addresses as 
>>> When the source or destination operand is a memory operand, the operand 
>>> must be aligned on a 16-byte boundary or a general-protection exception 
>>> (#GP) is generated
>> 
>> written in docs. When crashes, ESI contains value like `0x27DB63A8`, so it 
>> doesn’t seem aligned to 16-byte boundary. The line "`siginfo: 
>> ExceptionCode=0xc0000005, reading address 0xffffffff`" from `hs_err` file 
>> implicitly says it is GP, not a real "reading address 0xffffffff".
>> 
>> It might be related to clang-cl bug, see 
>> https://github.com/llvm/llvm-project/issues/55844
>> 
>> The workaround is to disable SSE when building 32bit on Windows. (`-mno-sse`)
>
> Roman Marchenko has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Update WebKitCompilerFlags.cmake

Looks good to me. basic sanity test ok

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

Marked as reviewed by jbhaskar (Committer).

PR Review: https://git.openjdk.org/jfx/pull/1764#pullrequestreview-2750686326

Reply via email to