<html><body><div dir="ltr"><div style="">
</div><div style=""><div>
<div>Hello David.</div><div dir="ltr"><br></div><div dir="ltr">Thanks for the hint (GitHub search seems to swallow that)</div><div dir="ltr"><br></div><div dir="ltr">(Is x86_32.ab used in 64Bit builds?)</div><div dir="ltr"><br></div><div dir="ltr">Btw regarding my initial question, the only requirement I have seen so far are asserts asking for SSE2 — since those instructions are part of the x86_64 baseline API it looks like the source at least does not require x86_64-v2 or newer. (That was probably an entirely different thing for x86).</div><div dir="ltr"><br></div><div dir="ltr">Gruss</div><div dir="ltr">Bernd</div><div id="ms-outlook-mobile-signature"><div style="direction:ltr">-- </div><div style="direction:ltr">http://bernd.eckenfels.net</div></div>
</div>
<div> </div><hr style="display:inline-block;width:98%" tabindex="-1"><div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif"><b>Von:</b> David Holmes <david.holmes@oracle.com><br><b>Gesendet:</b> Freitag, August 25, 2023 3:57 AM<br><b>An:</b> Bernd Eckenfels <ecki@zusammenkunft.net>; hotspot-dev@openjdk.org <hotspot-dev@openjdk.org><br><b>Betreff:</b> Re: Older unused Intel platform check functions<div> </div></font></div>Hi Bernd,
<br>
<br>On 25/08/2023 9:23 am, Bernd Eckenfels wrote:
<br>> Hello,
<br>>
<br>> I was checking out to see a hint of what’s the minimum microarchitecture
<br>> is, which makes sense to require for OpenJDK. I was not very successful
<br>> and it probably depends on compiler toolchain and distribution. Does
<br>> somebody have some insights?
<br>>
<br>> Having said that I noticed the following:
<br>>
<br>> The 20 years old netburst architecture has a test function but it’s not
<br>> used to make feature decisions in the JDK, it is just used to construct
<br>> a SSE feature string, where it can be safely removed (the feature flags
<br>> are reported anyway, worst case use cpu_family_description instead?)
<br>>
<br>> VM_Version::is_netburst
<br>>
<br>> If you want to keep it, it’s probably more consistent to rename it to
<br>> is_intel_netburst
<br>
<br>Yes netburst seem unused now.
<br>
<br>>
<br>> VM_Version::is_default_intel_cascadelake
<br>> VM_Version::is_intel_cascadelake
<br>>
<br>> Those two seem not to be used anywhere, can probably be removed as well.
<br>
<br>Yes they are used: ./cpu/x86/x86.ad ./cpu/x86/x86_64.ad
<br>
<br>Cheers,
<br>David
<br>
<br></div></div></body></html>