<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Apr 3, 2013, at 6:59 PM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;"><blockquote type="cite">Debug Info: according to DWARF 2, FORM_ref_addr the same size as an address on<br>the target system.<br><br>It was hard-coded to 4 bytes before. I can't get llvm to generate a<br>ref_addr on a reasonably sized testing case.<br><br><a href="rdar://problem/13559431">rdar://problem/13559431</a><br><br></blockquote><br>Unfortunately our current version really isn't 2, there are many<br>dwarf4-isms in our current output. If you'd like to have a dwarf-2<br>compatibility mode then you'll need to submit a patch that delineates<br>that through all of compilation.<br></div></blockquote><div><br></div>The DWARF_VERSION is set to 2 in llvm/Support/Dwarf.h so the header of our object file will say version 2.</div><div>Also our llvm dwarf parser does not check DWARF version and will parse ref_addr according to version 2.</div><div><div>And here it seems that we only support version 2 and 3.</div><div><div> static bool isSupportedVersion(unsigned version) {</div><div> return version == 2 || version == 3;</div><div> }</div></div><div><br></div></div><div>I know there are supports of dwarf4 stuff in our current debug info.</div><div>Are you okay with me changing the DWARF_VERSION to 4?</div><div>It may break a lot of things. </div><div>Another way is to enable this change only for darwin gdb.</div><div><br></div><div>Manman</div><div><br></div><div><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;"><br>Please revert this patch for now and we can discuss this in this<br>thread if you'd like.<br><br>-eric</div></blockquote></div><br></body></html>