<html>
<head>
<base href="http://llvm.org/bugs/" />
</head>
<body><table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>Bug ID</th>
<td><a class="bz_bug_link
bz_status_NEW "
title="NEW --- - [Reassociate] Use localized Changed flag to avoid calls to RewriteExprTree"
href="http://llvm.org/bugs/show_bug.cgi?id=22578">22578</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>[Reassociate] Use localized Changed flag to avoid calls to RewriteExprTree
</td>
</tr>
<tr>
<th>Product</th>
<td>libraries
</td>
</tr>
<tr>
<th>Version</th>
<td>trunk
</td>
</tr>
<tr>
<th>Hardware</th>
<td>PC
</td>
</tr>
<tr>
<th>OS</th>
<td>Linux
</td>
</tr>
<tr>
<th>Status</th>
<td>NEW
</td>
</tr>
<tr>
<th>Severity</th>
<td>normal
</td>
</tr>
<tr>
<th>Priority</th>
<td>P
</td>
</tr>
<tr>
<th>Component</th>
<td>Scalar Optimizations
</td>
</tr>
<tr>
<th>Assignee</th>
<td>unassignedbugs@nondot.org
</td>
</tr>
<tr>
<th>Reporter</th>
<td>mcrosier@codeaurora.org
</td>
</tr>
<tr>
<th>CC</th>
<td>llvmbugs@cs.uiuc.edu
</td>
</tr>
<tr>
<th>Classification</th>
<td>Unclassified
</td>
</tr></table>
<p>
<div>
<pre>The Reassociation pass uses a global flag to determine if something in the IR
has changed. The flag should be maintained per expression to avoid calls to
RewriteExprTree.
If the expression hasn't changed in some non-trivial way, then the rewriting is
unnecessary. Rewriting an unchanged expressions increases compile-time, drops
unsigned/signed wrap flags and generally results in a completely different
expression topology. I'm seeing that the rewriting generally produces the
longest critical path, which is especially bad for FP code.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are on the CC list for the bug.</li>
</ul>
</body>
</html>