<html>
<head>
<base href="https://bugs.llvm.org/">
</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 - [x86] Revert r319777(clang) and r319778(llvm) and r319911(llvm) from 6.0 branch"
href="https://bugs.llvm.org/show_bug.cgi?id=36357">36357</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>[x86] Revert r319777(clang) and r319778(llvm) and r319911(llvm) from 6.0 branch
</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>Windows NT
</td>
</tr>
<tr>
<th>Status</th>
<td>NEW
</td>
</tr>
<tr>
<th>Severity</th>
<td>enhancement
</td>
</tr>
<tr>
<th>Priority</th>
<td>P
</td>
</tr>
<tr>
<th>Component</th>
<td>Backend: X86
</td>
</tr>
<tr>
<th>Assignee</th>
<td>unassignedbugs@nondot.org
</td>
</tr>
<tr>
<th>Reporter</th>
<td>craig.topper@gmail.com
</td>
</tr>
<tr>
<th>CC</th>
<td>llvm-bugs@lists.llvm.org
</td>
</tr></table>
<p>
<div>
<pre>These commits removed some intrinsics and implemented them with native IR. But
it turns out that we may have messed up the operand order in the upgrade due to
inconsistent documentation from Intel.
Intel documentation says this for kunpackb intrinsic
k[7:0] := b[7:0]
k[15:8] := a[7:0]
k[MAX:16] := 0
But for kunpackw it says
k[31:0] := a[31:0]
k[63:32] := b[31:0]
k[MAX:64] := 0
Notice the order of 'a' and 'b' is reversed here. kunpackd is documented
similarly to kunpackw.
It turns out icc, gcc, and clang 5.0 all implement the order specified for
kunpackb for all 3 intrinsics.
It seems that when the upgrade was done in 6.0, we followed the
kunpackw/kunpackd documentation instead.
There have been other changes to the autoupgrade here after the 6.0 branch. So
rather than try to fix the 6.0 implementation and the trunk implementation
separately, I propose that we revert the commits on the 6.0 branch.</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>