[all-commits] [llvm/llvm-project] 7f51a2: [lldb][Language] Simplify SourceLanguage::GetDescr...

[Michael Buch via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7f51a2a47d2e706d04855b0e41690ebafa2b3238
      https://github.com/llvm/llvm-project/commit/7f51a2a47d2e706d04855b0e41690ebafa2b3238
  Author: Michael Buch <michaelbuch12 at gmail.com>
  Date:   2025-10-03 (Fri, 03 Oct 2025)

  Changed paths:
    M lldb/source/Target/Language.cpp
    M lldb/unittests/Target/CMakeLists.txt
    A lldb/unittests/Target/Language.cpp

  Log Message:
  -----------
  [lldb][Language] Simplify SourceLanguage::GetDescription (#161804)

Currently we don't benefit from the user-friendly names that
`LanguageDescription` returns because we would always use
`Language::GetNameForLanguageType`. I'm not aware of a situation where
`GetDescription` should prefer the non-human readable form of the name
with. This patch removes the call to `GetNameForLanguageType`.

`LanguageDescription` already handles languages that it doesn't know
about. For those it would return `Unknown`. The LLDB language types
should all be available via DWARF. If there are languages that don't map
cleanly between `lldb::LanguageType` and `DW_LANG`, then we should add
explicit support for that in the `SourceLanguage::SourceLanguage`
constructor.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications