[PATCH] D59008: [AMDGPU] Switch default dwarf version to 5

Mon Mar 25 17:04:08 PDT 2019

dblaikie added a comment.

In D59008#1442256 <https://reviews.llvm.org/D59008#1442256>, @t-tye wrote:

> In D59008#1442014 <https://reviews.llvm.org/D59008#1442014>, @dblaikie wrote:
>
> > In D59008#1441903 <https://reviews.llvm.org/D59008#1441903>, @t-tye wrote:
> >
> > > LGTM
> > >
> > > Do we know the state of split DWARF and DWARF compression for DWARF 5 (compared to DWARF 2)?
> >
> >
> > State of them in what sense? Compression is pretty orthogonal to any DWARF version - it's more about the container (ELF, etc) you use. Split DWARF is non-standardly supported in pre-v5, and I think it's functioning in the standards conformant v5 mode too.
>
>
> I had heard mention that at least split DWARF may not be fully supported yet for DWARF 5 in LLVM. But maybe that is old news:-)

Might be - nothing I know of right now. (type units aren't fully supported in DWARFv5 - but that's the only big thing on my list)

Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59008/new/

https://reviews.llvm.org/D59008