<html>
<head>
<base href="https://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 --- - [ppc] inefficient code generated for std::max(float, float)"
href="https://llvm.org/bugs/show_bug.cgi?id=26445">26445</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>[ppc] inefficient code generated for std::max(float, float)
</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>Backend: PowerPC
</td>
</tr>
<tr>
<th>Assignee</th>
<td>unassignedbugs@nondot.org
</td>
</tr>
<tr>
<th>Reporter</th>
<td>carrot@google.com
</td>
</tr>
<tr>
<th>CC</th>
<td>llvm-bugs@lists.llvm.org
</td>
</tr>
<tr>
<th>Classification</th>
<td>Unclassified
</td>
</tr></table>
<p>
<div>
<pre>Compile following code with options: $ ~/llvm/obj2/bin/clang++
--target=powerpc64le-grtev4-linux-gnu -m64 -mvsx -mcpu=power8 -O2 -c -o t9.o
t9.cc -fno-unroll-loops
#include <algorithm>
float foo(float* input, int s) {
float max_value = input[0];
for (int j = 1; j <= s; ++j)
max_value = std::max(max_value, input[j]);
return max_value;
}
I got following code for the loop body
.LBB0_2: # %for.body
# =>This Inner Loop Header: Depth=1
lfsu 0, 4(3)
fcmpu 0, 1, 0
isel 5, 3, 4, 0
lwz 5, 0(5)
mtvsrd 34, 5
stw 5, -12(1)
xxsldwi 13, 34, 34, 1
xscvspdpn 1, 13
bdnz .LBB0_2
There are several problems in this code snippet
1. instead of compare and choose maximum float value, the generated code uses
isel to choose the address of larger value, then load it into integer register,
then move it to fp register, then expand it to double type.
2. no need to store the max value to memory.
It causes one of our internal applications more than 2x slower.</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>