<div class="gmail_quote">On Mon, Jun 20, 2011 at 11:01 AM, Devang Patel <span dir="ltr"><<a href="mailto:dpatel@apple.com">dpatel@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"><br>
On Jun 19, 2011, at 3:20 PM, Nathan Jeffords wrote:<br>
<br>
> As it happens, I have created a patch that enables DWARF debugging on windows. It is against trunk revision 133401. I have tested using clang to compile, and mingw to link & debug.<br>
<br>
</div>+#include "../Target/X86/X86FixupKinds.h"<br></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><br>
This indicates layering violation to me.<br><br></blockquote><div><br></div><div>I agree with you on this, but after a little digging around, I don't see a good place to put the dependency on reloc_coff_secrel32. I couple of options jump to mind, but I don't understand the design well enough to know which is better.</div>
<div><br></div><div>We could generalize section relative fix-ups by adding a set to the MCFixupKind enumeration, then only support the 4 byte version for the COFF object writer.</div><div><br></div><div>Another option would be to add a method to MCCodeEmitter to request that it generate the required fix-up, as it currently seems to be the only piece of code to produce target specific fix-ups.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
It looks good otherwise. Do you have a testcase ?<br></blockquote><div><br></div><div>I do not have any test case yet, but I will produce some. This is pretty much the same patch as the one I submitted several months ago. It was reverted due to causing some build-bot failures. I imagine this will produce the same problems. I don't know how to go about trouble shooting those issues.</div>
<div><br></div><div>-Nathan</div><div><br></div></div>