<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
font-size:10.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
span.EmailStyle19
{mso-style-type:personal-reply;
font-family:"Calibri",sans-serif;
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-size:10.0pt;
mso-ligatures:none;}
@page WordSection1
{size:612.0pt 792.0pt;
margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
{page:WordSection1;}
--></style>
</head>
<body lang="EN-CA" link="blue" vlink="purple" style="word-wrap:break-word">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt">If there are no concerns with the JFR event, it’s initial settings or the test, given that the small compiler changes have already been reviewed would someone be kind enough to review and sponsor the commit<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">Thanks in advance<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">Mat<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">From: hotspot-jfr-dev <hotspot-jfr-dev-retn@openjdk.org> on behalf of Mat Carter <macarte@openjdk.org><br>
Date: Wednesday, October 18, 2023 at 11:12 AM<br>
To: hotspot-dev@openjdk.org <hotspot-dev@openjdk.org>, hotspot-jfr-dev@openjdk.org <hotspot-jfr-dev@openjdk.org><br>
Subject: Re: RFR: 8317562: [JFR] Compilation queue statistics [v2]<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt">On Wed, 18 Oct 2023 17:35:59 GMT, Vladimir Kozlov <kvn@openjdk.org> wrote:<br>
<br>
>> Mat Carter has updated the pull request incrementally with one additional commit since the last revision:<br>
>> <br>
>> fixed return type and changed NULL to nullptr<br>
><br>
> src/hotspot/share/compiler/compileBroker.cpp line 530:<br>
> <br>
>> 528: return _c2_compile_queue;<br>
>> 529: }<br>
>> 530: <br>
> <br>
> Note, `*_compiler_queue` could be `nullptr` if VM is build without C2 or C1 or when run with `-XX:-TieredCompilation` (only C2 is used) or with `-XX:TierdStopAtLevel={1,2,3}` (only C1 is used).<br>
> <br>
> Make sure you check it in JFR event.<br>
<br>
Thank you! The JFR event does check for NULL (now nullptr)<br>
<br>
-------------<br>
<br>
PR Review Comment: <a href="https://git.openjdk.org/jdk/pull/16211#discussion_r1364317003">
https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.openjdk.org%2Fjdk%2Fpull%2F16211%23discussion_r1364317003&data=05%7C01%7Cmatthew.carter%40microsoft.com%7C4c80af72893e4fb6326408dbd005cf39%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C638332495614878626%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=5wvfj1sZg3QJyoivoKGyz7xD3gO1lG3wRhuqFpt7sJY%3D&reserved=0</a><o:p></o:p></span></p>
</div>
</body>
</html>