[Lldb-commits] [PATCH] D143061: [lldb][Language] Add more language types

[Adrian Prantl via Phabricator via lldb-commits]

aprantl added inline comments.


================
Comment at: lldb/include/lldb/lldb-enumerations.h:492
+  eLanguageTypeAda2005 = 0x002e,
+  eLanguageTypeAda2012 = 0x002f,
+
----------------
Michael137 wrote:
> aprantl wrote:
> > Would it make sense to generate this list from the macros in `llvm/include/llvm/BinaryFormat/Dwarf.def` with some clever application of the ## operator?
> The only thing that's stopping us from doing this is that the constants for the vendor extensions are not consecutive with the rest of the constants. So if one ever does try to use that language constant we'd run into out-of-bounds accesses here and there
Do have arrays that are indexed by language? Or what would be an example of that?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143061