<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 - Copying of new LLVMContext::MD_access_group metadata missing in combineStoreToNewValue()"
href="https://bugs.llvm.org/show_bug.cgi?id=40117">40117</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>Copying of new LLVMContext::MD_access_group metadata missing in combineStoreToNewValue()
</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>enhancement
</td>
</tr>
<tr>
<th>Priority</th>
<td>P
</td>
</tr>
<tr>
<th>Component</th>
<td>Transformation Utilities
</td>
</tr>
<tr>
<th>Assignee</th>
<td>unassignedbugs@nondot.org
</td>
</tr>
<tr>
<th>Reporter</th>
<td>moritz.kreutzer@siemens.com
</td>
</tr>
<tr>
<th>CC</th>
<td>llvm-bugs@lists.llvm.org
</td>
</tr></table>
<p>
<div>
<pre>Hi,
I had a look at the recently merged commit for adding the MD_access_group
metadata globally: <a href="https://reviews.llvm.org/rL349725">https://reviews.llvm.org/rL349725</a>
In InstCombine's combineLoadToNewType() method, the new metadata gets copied to
the new load instruction alongside of the old MD_mem_parallel_loop_access
metadata. However, in the seemingly very similar combineStoreToNewValue()
method (which I assume does the same thing for stores instead of loads), the
new metadata is not copied to the new store instruction, but it seems to me as
if it should (see InstCombineLoadStoreAlloca.cpp:555). While I'm not 100% sure
that this is a bug, it seems slightly suspicious to me. I'd be happy to be
proven wrong and this is a WONTFIX, though.
Moritz</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>