<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, May 19, 2017 at 6:59 PM Eric Beckmann via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ecbeckmann added inline comments.<br>
<br>
<br>
================<br>
Comment at: llvm/include/llvm/Object/WindowsResource.h:59<br>
+ public:<br>
+  ~WindowsResource() override;<br>
+<br>
----------------<br>
ruiu wrote:<br>
> Do you need this?<br>
Yes because the destructor is declared virtual in the base class.<br><br></blockquote><div><br></div><div>I think what he meant is that the definition of the function doesn't do anything, it's just defined in the cpp file with =default.  So if that's the case, you can just delete it here. </div></div></div>