<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>Issue</th>
<td>
<a href=https://github.com/llvm/llvm-project/issues/66751>66751</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>
Large compile time regression with new MatchTable combiner
</td>
</tr>
<tr>
<th>Labels</th>
<td>
llvm:globalisel
</td>
</tr>
<tr>
<th>Assignees</th>
<td>
</td>
</tr>
<tr>
<th>Reporter</th>
<td>
aemerson
</td>
</tr>
</table>
<pre>
It seems we forgot to measure the compile time impact of the new MT combiner. When it was enabled for AArch64 in July it resulted in a large -Os 9% regression on CTMark/sqlite3 (and others too but I'm just using sqlite3 as a test case). Some time soon afterwards that regression went down to 3-4% but I haven't identified the commit responsible for that improvement.
However, even 3% is a very large CT regression.
The measurements were taken using a release + noasserts build of clang, without LTO/PGO on trunk.
Unfortunately some quick profiling doesn't show much except that `ExecuteMatchTable()` is taking most of the time in the combiner, which isn't surprising.
</pre>
<img width="1px" height="1px" alt="" src="http://email.email.llvm.org/o/eJxUk0FzIjcQhX-NuHSZGjQwMIc5eL0h2dS6nKqQyrln1Iy0aCRW3YL1v09pMI59gSrU9Ov3vW5kdmMg6tTmi9p8XWAWG1OHNFHiGBZ9NK_dNwEmmhiuBMeYxiggESZCzolALMEQp7PzBOImAjedcRCIx_kp0BWeD6Wid4HSEv61FMAJXJGBAvaeTOkKj49psM0aXIA_s38tJYk4eyFTfkPwmEaChxeGVukNJBoTMbsYIAZ4OjxjOim955_eCdWg9A6DgSiWEoPECH0W-Kb0doIfmQUyuzDCvRwZEIRYYEAmpdsl_B2nN0ccYwA8CqUrJsMgFuWj_pWCgInXULjUD-sy3qwGFi8UlN4KOENB3NGRuRObbg7PMbDr_Yz21tlN5xQvNFGQpaq-qurx9vlHvNKFktJPQBcKUBcdVwa_UHp94_N0-DDZp78fLN1DK61LnCU-PFF4g4GQyBMygdJfIERkpiQMfXbelEAHj2Es-lcnNmaB74cXpfd__f5SMpCUw-mT5D_hGJPkgEL-FbgA_ZndcIJzikfni6aJxDdCbOMVpjxYoF8DneUGQzXVb79oyELPKIM9lIVReqd0q5qquBc8lT5T5Pedu61huIOeF2-e2rrBgrvr5XROrhhfLkxXm7ZucUHdqmk3q1Vdr5qF7dphtT2ut4NGwupozEbXpt5WhCtsBzPsFq7Tla6rdtVWu_WubpZtfTTNChu9M9Vug61aVzSh80vvL9MypnHhmDN1TbPdrBYee_I8X5_WpULVj6OPPXrH5JXW5ShTV14e-jyyWlfesfD_3cSJp-77nP2nM_y4n07s7RDfEb5zWeTkOytyZlU_Kr1Xej86sblfDnFSej8Pdft6OKf4gwZRej97YKX3s43_AgAA__8fqHNZ">