
<html>


<head>


<meta http-equiv="Content-Type" content="text/html; charset=utf-8">


</head>


<body style="overflow-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;">


<br id="lineBreakAtBeginningOfMessage">


<div><br>


<blockquote type="cite">


<div>On Oct 17, 2024, at 4:45 AM, Gary Frost <gary.frost@oracle.com> wrote:</div>


<br class="Apple-interchange-newline">


<div>


<div class="elementToProof" style="font-style: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt;">


Paul,<br>


<br>


This looks reasonable.<br>


<br>


Debugging this my end yesterday, I concluded that where the method is supposed to return a value and I see a return without a value,  it should be a huge hint to my C99 code gen that this return is not reachable and so my C99 code gen can just omit the return. <br>


<br>


</div>


</div>


</blockquote>


<blockquote type="cite">


<div>


<div class="elementToProof" style="font-style: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt;">


In the case where the return is void I would indeed emit an unnecessary 'return;' which thankfully CUDA and OpenCL just accepts as noise.<br>


<br>


I was about to make this mildy hacky change this morning, but now will await the fix you suggest, as I do think tagging this with an unreachable Op rather than return Op would be far more useful in the general case.<br>


<br>


</div>


</div>


</blockquote>


<div><br>


</div>


Should be ready by the end of this week.</div>


<div><br>


</div>


<div><br>


<blockquote type="cite">


<div>


<div class="elementToProof" style="font-style: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt;">


BTW outlook tagged the email below as in 'Slovak' 😉<br>


<br>


</div>


</div>


</blockquote>


<div><br>


</div>


:-) my email client was convinced I was referring to email addresses in the content that were not present in to the to/cc fields.</div>


<div><br>


</div>


<div>Paul.</div>


</body>


</html>