<div dir="ltr">lgtm<div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le lun. 27 févr. 2023 à 17:17, Jayathirth D V <<a href="mailto:jdv@openjdk.org">jdv@openjdk.org</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> Under <a href="https://bugs.openjdk.org/browse/JDK-8264846" rel="noreferrer" target="_blank">https://bugs.openjdk.org/browse/JDK-8264846</a> we moved to -O3 level of gcc optimizations from -O1 level for libawt build. This improved our J2DBench performance numbers in some options considerably.<br>
> <br>
> Recent changes done under <a href="https://bugs.openjdk.org/browse/JDK-8299337" rel="noreferrer" target="_blank">https://bugs.openjdk.org/browse/JDK-8299337</a> causes difference in generated code by gcc and this is resulting in performance regression for bimg_misc-* J2DBench options in our performance servers. Under <a href="https://bugs.openjdk.org/browse/JDK-8299337" rel="noreferrer" target="_blank">https://bugs.openjdk.org/browse/JDK-8299337</a> we have just removed unused variables and it is a cleanup task.<br>
> <br>
> We can force gcc to generate position independent code by using -fpic option.Also i have removed -fgcse-after-reload option for gcc, because this is by default covered under -O3 level of optimization introduced under <a href="https://bugs.openjdk.org/browse/JDK-8264846" rel="noreferrer" target="_blank">https://bugs.openjdk.org/browse/JDK-8264846</a>.<br>
> <br>
> With this change bimg_misc-* J2DBench option performance regression is resolved and there are no regression in other options of J2DBench or SwingMark and it is verified in our performance servers.<br>
<br>
Jayathirth D V has updated the pull request incrementally with one additional commit since the last revision:<br>
<br>
  Update comment<br>
<br>
-------------<br>
<br>
Changes:<br>
  - all: <a href="https://git.openjdk.org/jdk/pull/12761/files" rel="noreferrer" target="_blank">https://git.openjdk.org/jdk/pull/12761/files</a><br>
  - new: <a href="https://git.openjdk.org/jdk/pull/12761/files/0c39e4c3..e8d25674" rel="noreferrer" target="_blank">https://git.openjdk.org/jdk/pull/12761/files/0c39e4c3..e8d25674</a><br>
<br>
Webrevs:<br>
 - full: <a href="https://webrevs.openjdk.org/?repo=jdk&pr=12761&range=01" rel="noreferrer" target="_blank">https://webrevs.openjdk.org/?repo=jdk&pr=12761&range=01</a><br>
 - incr: <a href="https://webrevs.openjdk.org/?repo=jdk&pr=12761&range=00-01" rel="noreferrer" target="_blank">https://webrevs.openjdk.org/?repo=jdk&pr=12761&range=00-01</a><br>
<br>
  Stats: 3 lines in 1 file changed: 0 ins; 2 del; 1 mod<br>
  Patch: <a href="https://git.openjdk.org/jdk/pull/12761.diff" rel="noreferrer" target="_blank">https://git.openjdk.org/jdk/pull/12761.diff</a><br>
  Fetch: git fetch <a href="https://git.openjdk.org/jdk" rel="noreferrer" target="_blank">https://git.openjdk.org/jdk</a> pull/12761/head:pull/12761<br>
<br>
PR: <a href="https://git.openjdk.org/jdk/pull/12761" rel="noreferrer" target="_blank">https://git.openjdk.org/jdk/pull/12761</a><br>
</blockquote></div>