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

[Adrian Prantl via Phabricator via llvm-commits]

aprantl added a comment.

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.


https://reviews.llvm.org/D47811