<!DOCTYPE html>
<html data-lt-installed="true">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body style="padding-bottom: 1px;">
<p>Hi, Sergey,<br>
</p>
<div class="moz-cite-prefix">On 15.08.2024 16:31, Sergey Kaplun
wrote:<br>
</div>
<blockquote type="cite" cite="mid:Zr4DF7MMqiRSVOfr@root">
<pre class="moz-quote-pre" wrap="">Sergey,
On 15.08.24, Sergey Bronnikov wrote:
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">Hi, Sergey
Thanks for the patch! See my comments below.
On 14.08.2024 16:55, Sergey Kaplun wrote:
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">This patch moves the <alias_alloc.lua> test from the <misc> to the
created <opt/mem> directory, includes it in <index>, and names subtests.
Testing optimizations in <lj_opt_mem.c> requires CSE, DCE, FOLD, and FWD
optimizations enabled. Hence, the corresponding flags are used for the
<mem> directory in <index>.
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
It is still unclear for me *why* a reason of this change.
</pre>
</blockquote>
</blockquote>
The main idea behind the patch is "getting rid of <misc>
suite".<br>
<blockquote type="cite" cite="mid:Zr4DF7MMqiRSVOfr@root">
<pre class="moz-quote-pre" wrap="">
The tests for other optimizations also require the necessary flags, see
<index>. Also, I don't understand what *change* you mean -- it is
enabling of test, not changing its functionality.
</pre>
<blockquote type="cite">
<pre class="moz-quote-pre" wrap="">
</pre>
</blockquote>
<pre class="moz-quote-pre" wrap="">
<snipped>
</pre>
</blockquote>
</body>
<lt-container></lt-container>
</html>