<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 - performance-no-automatic-move should accept a list of classes to be ignored"
href="https://bugs.llvm.org/show_bug.cgi?id=48464">48464</a>
</td>
</tr>
<tr>
<th>Summary</th>
<td>performance-no-automatic-move should accept a list of classes to be ignored
</td>
</tr>
<tr>
<th>Product</th>
<td>clang
</td>
</tr>
<tr>
<th>Version</th>
<td>11.0
</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>Tooling
</td>
</tr>
<tr>
<th>Assignee</th>
<td>unassignedclangbugs@nondot.org
</td>
</tr>
<tr>
<th>Reporter</th>
<td>aacid@kde.org
</td>
</tr>
<tr>
<th>CC</th>
<td>llvm-bugs@lists.llvm.org
</td>
</tr></table>
<p>
<div>
<pre>I see the point for some classes, but take for example Qt classes like QString
that is refcounted, returning it by copy is as good as returning it by move
because all that happens is that the internal refcounter is updated.
But if if you have a long function with
const QString foo = ...
// lots of code
return foo;
it's worth foo being const since it won't be slower than if it wasn't and
prevents you making the mistake of changing foo.
That's why i think it'd be good if there was a way to tell
performance-no-automatic-move to ignore some classes from the check.</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>