<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=utf-8">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Here is a real patch.<br>
<div class="moz-forward-container"><br>
<br>
-------- Weitergeleitete Nachricht --------
<table class="moz-email-headers-table" cellpadding="0"
cellspacing="0" border="0">
<tbody>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">Betreff:
</th>
<td>Bug in Support/Allocator.h</td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">Datum: </th>
<td>Mon, 11 May 2015 17:25:45 +0200</td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">Von: </th>
<td>Kal <a class="moz-txt-link-rfc2396E" href="mailto:b17c0de@gmail.com"><b17c0de@gmail.com></a></td>
</tr>
<tr>
<th align="RIGHT" nowrap="nowrap" valign="BASELINE">An: </th>
<td>LLVM Developers Mailing List (<a class="moz-txt-link-abbreviated" href="mailto:llvmdev@cs.uiuc.edu">llvmdev@cs.uiuc.edu</a>)
<a class="moz-txt-link-rfc2396E" href="mailto:llvmdev@cs.uiuc.edu"><llvmdev@cs.uiuc.edu></a></td>
</tr>
</tbody>
</table>
<br>
<br>
<pre>Hi,
llvm::BumpPtrAllocatorImpl::Reset() looks like it has a bug.
There is this check at the top:
if (Slabs.empty())
return;
But what if you don't have any normal Slabs allocated but only
CustomSizedSlabs.
I think this should be:
if (Slabs.empty() && CustomSizedSlabs.empty())
return;
</pre>
<br>
</div>
<br>
</body>
</html>