<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 9, 2016 at 6:11 PM, Matt Arsenault <span dir="ltr"><<a href="mailto:Matthew.Arsenault@amd.com" target="_blank">Matthew.Arsenault@amd.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 03/09/2016 05:21 PM, David Blaikie wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
What's the reason for marking the classes final? We don't usually go out of our way to do that, except for -Wnon-virtual-dtor silencing (in my experience/practice) - so perhaps it's that case? (just not clear to me from the commit/code (without going & looking at the hierarchies, etc)<br>
</blockquote></span>
Because they can be. There's no reason to further subclass these, and might as well allow the compiler to emit direct calls to them.</blockquote><div><br>I don't think we've really made any habit of that across LLVM - I'd be slightly hesitant to bother without some evidence that it helps, but it's just a single keyword, so I'm not terribly fussed if you prefer it that way.<br><br>- Dave<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888"><br>
<br>
-Matt<br>
</font></span></blockquote></div><br></div></div>