[PATCH] D47811: [PATCH] [DWARF] Kotlin language

[Vasily Levchenko via Phabricator via llvm-commits]

vvlevchenko added a comment.

In https://reviews.llvm.org/D47811#1273040, @aprantl wrote:

> In https://reviews.llvm.org/D47811#1272985, @vvlevchenko wrote:
>
> > In https://reviews.llvm.org/D47811#1272883, @aprantl wrote:
> >
> > > I think a patch that adds the missing constants to Dwarf.def would be sufficient. This is mostly for documentation purposes to avoid accidentally re-use ranges used by other compilers.
> >
> >
> > Could you please elaborate more? I still doesn't understand which constants are missed and how I can get them. Actually I have got only one assigned by dwarfstd  http://www.dwarfstd.org/ShowIssue.php?issue=170503.1.
> >
> > > It's possible that libdwarf (http://wiki.dwarfstd.org/index.php?title=Libdwarf_And_Dwarfdump) also defines additional extensions.
>
>
> Note that is is only tangentially related to your patch.
>
> Paul was asking that instead of using the tentative DWARF 6 definition, you'd allocate an LLVM vendor extension somewhere between `DW_LANG_lo_user` and `DW_LANG_hi_user`. From there we discussed how to make a safe choice, which prompted me to point out that the list of vendor extensions in LLVM isn't complete and that there are a couple of other sources to consult before picking a spare number. And I said it would be nice (but not necessary!) to submit a patch that adds other known vendor extensions to LLVM's Dwarf.def so this gets easier next time.


Thank you, for clarification.


https://reviews.llvm.org/D47811