<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</head>
<body style="overflow-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;">
That looks good. I recommend:
<div><br>
</div>
<div>- Removing the system property for <span class="pl-s">VECTOR_ACCESS_OOB_CHECK</span></div>
<div>- The benchmark methods should return the result (JMH will send it down a blackhole) <br>
<div>- You may also want to remove the allocation of the result matrix (pass it in as a pre-allocated argument and return it), since what we are interested here is how this all optimizes for the loop structure.</div>
<div><br>
</div>
<div>The process is:</div>
<div><br>
</div>
<div>1. Sign the OCA <a href="https://oca.opensource.oracle.com/">https://oca.opensource.oracle.com/</a></div>
<div>2. Submit a PR to <a href="https://github.com/openjdk/jdk/">https://github.com/openjdk/jdk/</a>, placing the class in test/micro/org/openjdk/bench/jdk/incubator/vector/</div>
<div>(Building the JDK and testing should be straightforward, for testing see <a href="https://github.com/openjdk/jdk/blob/master/doc/testing.md">https://github.com/openjdk/jdk/blob/master/doc/testing.md</a>)</div>
<div>3. I can review/sponsor</div>
<div><br>
</div>
<div>Paul.</div>
<div> <br>
<blockquote type="cite">
<div>On Aug 11, 2023, at 12:24 AM, Martin Stypinski <mstypinski@gmail.com> wrote:</div>
<br class="Apple-interchange-newline">
<div>
<div dir="ltr">
<div>Hi all,</div>
<div><br>
</div>
<div>I started this thing a long time ago and wanted to actually contribute my matrix benchmark as suggested by Paul. I made some changes:
<a href="https://gist.github.com/Styp/618cf9d139151727f46396235aff09a3">https://gist.github.com/Styp/618cf9d139151727f46396235aff09a3</a></div>
<div><br>
</div>
<div>Paul; What do you think? And how does the process work?</div>
<div><br>
</div>
<div>Best</div>
<div>Martin<br>
</div>
</div>
</div>
</blockquote>
</div>
<br>
</div>
</body>
</html>