<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Oct 18, 2010, at 4:52 PM, Chandler Carruth wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div class="gmail_quote">On Mon, Oct 18, 2010 at 4:47 PM, Bill Wendling <span dir="ltr"><<a href="mailto:wendling@apple.com">wendling@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Oct 18, 2010, at 4:18 PM, Chandler Carruth wrote:<br>
<br>
> Author: chandlerc<br>
> Date: Mon Oct 18 18:18:51 2010<br>
> New Revision: 116766<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=116766&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=116766&view=rev</a><br>
> Log:<br>
> Add a virtual destructor to silence a GCC warning.<br>
><br>
> Modified:<br>
> llvm/trunk/include/llvm/MC/MCObjectFormat.h<br>
><br>
> Modified: llvm/trunk/include/llvm/MC/MCObjectFormat.h<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/MCObjectFormat.h?rev=116766&r1=116765&r2=116766&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/MCObjectFormat.h?rev=116766&r1=116765&r2=116766&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/include/llvm/MC/MCObjectFormat.h (original)<br>
> +++ llvm/trunk/include/llvm/MC/MCObjectFormat.h Mon Oct 18 18:18:51 2010<br>
> @@ -15,6 +15,8 @@<br>
><br>
> class MCObjectFormat {<br>
> public:<br>
> + virtual ~MCObjectFormat() {}<br>
> +<br>
<br>
</div>Hi Chandler,<br>
<br>
Should the definition for this go in the MCObjectFormat.cpp file to "anchor" the vtable?<br></blockquote><div><br></div><div>I wasn't sure. I placed it here as that seemed like the minimal change from the original code (it is an abstract interface afterall). I'm happy to move it to the cpp file if anchoring the vtable (and forcing this call to be out-of-line) is desirable.</div>
</div>
</blockquote></div><br><div>As long as it complies with <a href="http://llvm.org/docs/CodingStandards.html#ll_virtual_anch">http://llvm.org/docs/CodingStandards.html#ll_virtual_anch</a> it will be okay. :)</div><div><br></div><div>-bw</div><div><br></div></body></html>