[lldb-dev] Acceptance of dwarf2/3 suggested form of DW_AT_comp_dir

[Greg Clayton]

Looks good. We can tweak it in the future if required if this ever fails.

> On Aug 22, 2014, at 4:57 AM, Matthew Gardiner <mg11 at csr.com> wrote:
> 
> Hi Greg,
> 
> I've modified the DWARF parsing accordingly. See:
> 
> http://reviews.llvm.org/D5022
> 
> 
> if you're happy with this, I'll  submit on tuesday. It resulted in no regressions when I ran the python tests on linux.
> 
> Matt
> 
> 
> Greg Clayton wrote:
>> On Aug 21, 2014, at 4:23 AM, Matthew Gardiner <mg11 at csr.com> wrote:
>>> Apologies for me getting noisy on this thread.
>>> 
>>> However, I don't think it's possible to work around toolchains that adopt the DWARF2/3 suggested form. The problem occurs if lldb wants to support debugging ELFs produced on different hosts - i.e. if the producer and consumers of the DWARF are either POSIX/Windows. We could then have a situation where a comp_dir string is something like:
>>> 
>>> c:/home/source
>> Actually I think you will be ok because it will be:
>> 
>> c:\home\source  for windows...
>> 
>>> in this case, we cannot determine whether c is a drive-letter or a hostname.
>>> 
>>> (So, I think we'll be changing our toolchain accordingly :-)
>>> 
>>> However, if anyone has any opinions on this DWARF2/3 debate I'd be interested in hearing them.
>> Feel free to modify the DWARF parser to strip off the leading hostname to get things fixed and working. If it is part of the older _or_ current spec, we should support it in LLDB.
>> 
>> Greg
>> 
>> 
>>  To report this email as spam click https://www.mailcontrol.com/sr/MZbqvYs5QwJvpeaetUwhCQ== .
> 
> 
> 
> Member of the CSR plc group of companies. CSR plc registered in England and Wales, registered number 4187346, registered office Churchill House, Cambridge Business Park, Cowley Road, Cambridge, CB4 0WZ, United Kingdom
> More information can be found at www.csr.com. Keep up to date with CSR on our technical blog, www.csr.com/blog, CSR people blog, www.csr.com/people, YouTube, www.youtube.com/user/CSRplc, Facebook, www.facebook.com/pages/CSR/191038434253534, or follow us on Twitter at www.twitter.com/CSR_plc.
> New for 2014, you can now access the wide range of products powered by aptX at www.aptx.com.