--0000000000005339b405c9fae593
Content-Type: text/plain; charset="UTF-8"

Just a small update:

On Windows we can also use Clang from MSYS. And it gives 30-40% better
performance in the benchmarks. See the attached spreadsheet for the details.

On Thu, Aug 19, 2021 at 10:11 AM Dmitry Kazakov <dimula73@gmail.com> wrote:

> Hi, all!
>
> I spent last week testing different compilers on Linux and Windows. I
> decided to gather all the information I found in a nice report so we could
> reuse it later :)
>
> Report about compilers available for Krita
> ==========================================
>
> Linux
> -----
>
> 1) Ubuntu 16.04
>
>    * uses GCC 5.5.0 by default
>    * has GCC 9.4.0 available via `ppa:ubuntu-toolchain-r/test` repository
>
>      - GCC 9.4.0 gives 10-30% better performance on Linux (I have tested
> on both, Intel and Ryzen7)
>      - I have already switched our AppImage builds to use GCC 9.4.0
>
> 2) Ubuntu 20.04
>
>    * uses GCC 9.3.0 by default
>    * has GCC 11.1.0 available via `ppa:ubuntu-toolchain-r/test` repository
>
>      - GCC 11.1.0 gives almost the same performance as 9.4.0 (about 2%
> better though)
>      - GCC 11.1.0 needs our deps patched a bit, because of changes in
> include header dependencies. In practice, you just need to add `#include
> <limits>` here and there.
>      - The patches that fix deps compilation on GCC 11.1.0 have already
> been pushed to master
>
> Windows
> -------
>
> 1) MinGW64-builds (the ones we use atm)
>
>    * uses GCC 7.3.0
>    * works somewhat fine
>
> 2) TDM-GCC (https://jmeubank.github.io/tdm-gcc/)
>
>    * uses GCC 10.3.0
>    * compiles Krita and its deps fine
>    * (!) causes speed regressions in the brush benchmarks
>    * (!) the compiler itself is 32-bit, so it might be not very efficient
> on 64-bit devices
>
> 3) MSYS (msys2.org)
>
>    * uses GCC 10.3.0
>    * the compiler is 64-bit, which is good
>    * when installing MSYS-MINGW64 one must skip installing GDB, because it
> requires Python, which conflicts with Krita's python
>    * (!) has three libraries dynamically linked to GCC, which conflict
> with Krita's deps:
>
>       - expat
>       - iconv
>       - zlib
>
>       Compiling of these deps should be manually patched out from Krita's
> deps (they are listed as deps to other deps)
>
>    * (!) drmingw fails to build (though I guess it is solvable)
>    * (!) compilation script of SIP and PyQt should be patched to use
> `--install-base` instead of `--prefix`. I don't know why.
>    * (!) still causes the same regressions in the brush benchmarks
>    * (!) when packaging, Krita package fails to run on another system
> because of the absent zlib, libintl and expat. Perhaps they should be added
> to teh package manually. I didn't try it though.
>
> Raw data of the benchmarks can be found here:
>
>
> https://docs.google.com/spreadsheets/d/1mvTC3Ny73_0F35MscUZBHyYCXRfoX06so5zI5LL2DqU
>
> --
> Dmitry Kazakov
>


-- 
Dmitry Kazakov

--0000000000005339b405c9fae593
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr">Just a small update:<div><br></div><div>On Windows we can =
also use Clang from MSYS. And it gives 30-40% better performance in the ben=
chmarks. See the attached spreadsheet for the details.</div></div><br><div =
class=3D"gmail_quote"><div dir=3D"ltr" class=3D"gmail_attr">On Thu, Aug 19,=
 2021 at 10:11 AM Dmitry Kazakov &lt;<a href=3D"mailto:dimula73@gmail.com">=
dimula73@gmail.com</a>&gt; wrote:<br></div><blockquote class=3D"gmail_quote=
" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);=
padding-left:1ex"><div dir=3D"ltr">Hi, all!<div><br></div><div>I spent last=
 week testing different compilers on Linux and Windows. I decided to gather=
 all the=C2=A0information I found in a nice report so we could reuse it lat=
er :)</div><div><br></div><div>Report about compilers available for Krita<b=
r>=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D<br><br>Linux<br>----=
-<br><br>1) Ubuntu 16.04<br>=C2=A0 =C2=A0<br>=C2=A0 =C2=A0* uses GCC 5.5.0 =
by default<br>=C2=A0 =C2=A0* has GCC 9.4.0 available via `ppa:ubuntu-toolch=
ain-r/test` repository<br><br>=C2=A0 =C2=A0 =C2=A0- GCC 9.4.0 gives 10-30% =
better performance on Linux (I have tested on both, Intel and Ryzen7)<br>=
=C2=A0 =C2=A0 =C2=A0- I have already switched our AppImage builds to use GC=
C 9.4.0<br><br>2) Ubuntu 20.04<br>=C2=A0 =C2=A0<br>=C2=A0 =C2=A0* uses GCC =
9.3.0 by default<br>=C2=A0 =C2=A0* has GCC 11.1.0 available via `ppa:ubuntu=
-toolchain-r/test` repository <br>=C2=A0 =C2=A0<br>=C2=A0 =C2=A0 =C2=A0- GC=
C 11.1.0 gives almost the same performance as 9.4.0 (about 2% better though=
)<br>=C2=A0 =C2=A0 =C2=A0- GCC 11.1.0 needs our deps patched a bit, because=
 of changes in include header dependencies. In practice, you just need to a=
dd `#include &lt;limits&gt;` here and there.<br>=C2=A0 =C2=A0 =C2=A0- The p=
atches that fix deps compilation on GCC 11.1.0 have already been pushed to =
master<br>	 <br>Windows<br>-------<br><br>1) MinGW64-builds (the ones we us=
e atm)<br>=C2=A0 =C2=A0<br>=C2=A0 =C2=A0* uses GCC 7.3.0<br>=C2=A0 =C2=A0* =
works somewhat fine<br>=C2=A0 =C2=A0<br>2) TDM-GCC (<a href=3D"https://jmeu=
bank.github.io/tdm-gcc/" target=3D"_blank">https://jmeubank.github.io/tdm-g=
cc/</a>)<br><br>=C2=A0 =C2=A0* uses GCC 10.3.0<br>=C2=A0 =C2=A0* compiles K=
rita and its deps fine <br>=C2=A0 =C2=A0* (!) causes speed regressions in t=
he brush benchmarks<br>=C2=A0 =C2=A0* (!) the compiler itself is 32-bit, so=
 it might be not very efficient on 64-bit devices<br>=C2=A0 =C2=A0<br>3) MS=
YS (<a href=3D"http://msys2.org" target=3D"_blank">msys2.org</a>)<br>=C2=A0=
 <br>=C2=A0 =C2=A0* uses GCC 10.3.0<br>=C2=A0 =C2=A0* the compiler is 64-bi=
t, which is good<br>=C2=A0 =C2=A0* when installing MSYS-MINGW64 one must sk=
ip installing GDB, because it requires Python, which conflicts with Krita&#=
39;s python<br>=C2=A0 =C2=A0* (!) has three libraries dynamically linked to=
 GCC, which conflict with Krita&#39;s deps:<br>=C2=A0 =C2=A0 =C2=A0<br>=C2=
=A0 =C2=A0 =C2=A0 - expat<br>=C2=A0 =C2=A0 =C2=A0 - iconv<br>=C2=A0 =C2=A0 =
=C2=A0 - zlib<br>	 <br>=C2=A0 =C2=A0 =C2=A0 Compiling of these deps should =
be manually patched out from Krita&#39;s deps (they are listed as deps to o=
ther deps)<br>	 <br>=C2=A0 =C2=A0* (!) drmingw fails to build (though I gue=
ss it is solvable)<br>=C2=A0 =C2=A0* (!) compilation script of SIP and PyQt=
 should be patched to use `--install-base` instead of `--prefix`. I don&#39=
;t know why.<br>=C2=A0 =C2=A0* (!) still causes the same regressions in the=
 brush benchmarks<br>=C2=A0 =C2=A0* (!) when packaging, Krita package fails=
 to run on another system because of the absent zlib, libintl and expat. Pe=
rhaps they should be added to teh package manually. I didn&#39;t try it tho=
ugh.<br><div><br></div><div>Raw data of the benchmarks can be found here:</=
div><div><br></div><div><a href=3D"https://docs.google.com/spreadsheets/d/1=
mvTC3Ny73_0F35MscUZBHyYCXRfoX06so5zI5LL2DqU" target=3D"_blank">https://docs=
.google.com/spreadsheets/d/1mvTC3Ny73_0F35MscUZBHyYCXRfoX06so5zI5LL2DqU</a>=
<br></div><div><br></div>-- <br><div dir=3D"ltr">Dmitry Kazakov</div></div>=
</div>
</blockquote></div><br clear=3D"all"><div><br></div>-- <br><div dir=3D"ltr"=
 class=3D"gmail_signature">Dmitry Kazakov</div>

--0000000000005339b405c9fae593--