<div dir="ltr"><div>Hi Magnus,</div><div><br></div><div>I put in a fix for another Win32 build break recently but hadn't realised it was broken again - ref <a href="https://bugs.openjdk.org/browse/JDK-8319958">https://bugs.openjdk.org/browse/JDK-8319958</a>. Since the Win32 port is currently only deprecated I was already wondering if we should go for the simple solution of having some github PR checks added for Win32 along with the other platforms (Obviously it would need the parameter to enable deprecated ports) as I feel this would help prevent things being merged and subsequently backported that break things. In the case of the one I fixed the PR that caused the build failure also got backported to the JDK17 codebase - for which there are no plans to deprecate - so potentially trapping things earlier would prevent such problems seeping through to earlier versions requiring subsequent remedial patches.<br></div><div><br></div><div>I think there's a discussion to be had here regarding 32-bit support in general going forward - I have a specific question about arm32 and whether anyone is actively maintaining that one but perhaps that's a matter for another thread unless anyone wants to jump in and say they'll be keeping arm32 alive in the codebase.<br></div><div><br></div><div><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div>Regards,</div><div><br></div><div>Stewart...</div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 5 Dec 2023 at 10:56, Magnus Ihse Bursie <<a href="mailto:magnus.ihse.bursie@oracle.com">magnus.ihse.bursie@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">I just built windows-x86, and noticed that the build is broken:<br>
<br>
src\hotspot\share\prims\downcallLinker.cpp(36): error C2373: <br>
'DowncallLinker::capture_state': redefinition; different type modifiers<br>
<br>
I think this is most likely caused by JDK-8254693, which was integrated <br>
three weeks ago. If no-one has discovered this, or bothered to fix it, <br>
maybe that is the signal we need to decide to actually stop pretending <br>
that windows-x86 is something we can build for, and just remove any <br>
remaining windows-x86 code?<br>
<br>
/Magnus<br>
<br>
<br>
</blockquote></div>