<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 - MCSymbolWasm is not trivially destructible"
href="https://bugs.llvm.org/show_bug.cgi?id=45452">45452</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>MCSymbolWasm is not trivially destructible
</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>MC
</td>
</tr>
<tr>
<th>Assignee</th>
<td>unassignedbugs@nondot.org
</td>
</tr>
<tr>
<th>Reporter</th>
<td>sbc@chromium.org
</td>
</tr>
<tr>
<th>CC</th>
<td>llvm-bugs@lists.llvm.org
</td>
</tr></table>
<p>
<div>
<pre>It seems that the MCSymbol class and its subclasses need to be trivially
destructible since MCContext doesn't destructors for each symbol during
::reset(). It only calls `Symbols.clear();`.
Currently there are 3 optional std::string fields that leak memory. Until
recently none of our test cases caught this (I believe) because all our tests
involved small string that didn't allocate. But recently this change
triggered an lsan warning: <a href="https://reviews.llvm.org/D77192">https://reviews.llvm.org/D77192</a>.
When we fix this we should add static asserts that these classes are trivially
destructible.</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>