<div dir="ltr">For the disassembler patch, the problem is you've defaulted the destructor but you've got a unique_ptr to a forward declared class.  MSVC is actually correct in failing to compile this, and I'm not sure why other compilers are accepting it.  My guess is something to do with the order of includes from the cpp file.<div><br></div><div>To fix this you need to remove the `default` keyword from the destructor and provide an empty implementation of the destructor in DisassemblerLLVMC.cpp</div><div><br></div><div>This problem occurs in other places too.  In OperatingSystemGo.h the same problem exists with DynamicRegisterInfo.  That's the one causing the operating system patch to fail.</div><div><br></div><div>What compiler are you testing this with?  This should fail under clang as well, I'm surprised it doesn't.  Are you using GCC by chance?</div><div><br></div><div>In any case, you'll need to fix all the occurrences of having a defaulted destructor in a class with a std::unique_ptr of an incomplete type.</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Oct 22, 2015 at 10:48 AM Eugene Zelenko <<a href="mailto:eugene.zelenko@gmail.com">eugene.zelenko@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Oct 22, 2015 at 10:09 AM, Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> wrote:<br>
> It looks like you're generating these diffs from an SVN repo, so I'm seeing<br>
> lldb/trunk in the paths.  Because of that, I can't apply it on my git repo<br>
> which doesn't have the "trunk" folder.  Can you generate the diffs for the 2<br>
> aforementioned CLs by cding into the trunk directory and generating the diff<br>
> there?<br>
<br>
See attached files. I did them in same way as diffs for Differential.<br>
</blockquote></div></div>