Skip to content

Commit 7020913

Browse files
committed
[lldb][Lanugage] Add Language::GetDisplayNameForLanguageType API
The intention for this API is to be used when presenting language names to the user, e.g., in expression evaluation diagnostics or LLDB errors. Most uses of `GetNameForLanguageType` can be probably replaced with `GetDisplayNameForLanguageType`, but that's out of scope of this PR. This uses `llvm::dwarf::LanguageDescription` under the hood, so we would lose the version numbers in the names. If we deem those to be important, we could switch to an implementation that hardcodes a list of user-friendly names with version numbers included.
1 parent faf070f commit 7020913

File tree

2 files changed

+11
-0
lines changed

2 files changed

+11
-0
lines changed

lldb/include/lldb/Target/Language.h

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -404,8 +404,15 @@ class Language : public PluginInterface {
404404
GetLanguageTypeFromString(const char *string) = delete;
405405
static lldb::LanguageType GetLanguageTypeFromString(llvm::StringRef string);
406406

407+
/// Returns the internal LLDB name for the specified language. When presenting
408+
/// the language name to users, use \ref GetDisplayNameForLanguageType
409+
/// instead.
407410
static const char *GetNameForLanguageType(lldb::LanguageType language);
408411

412+
/// Returns a user-friendly name for the specified language.
413+
static llvm::StringRef
414+
GetDisplayNameForLanguageType(lldb::LanguageType language);
415+
409416
static void PrintAllLanguages(Stream &s, const char *prefix,
410417
const char *suffix);
411418

lldb/source/Target/Language.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -270,6 +270,10 @@ const char *Language::GetNameForLanguageType(LanguageType language) {
270270
return language_names[eLanguageTypeUnknown].name;
271271
}
272272

273+
llvm::StringRef Language::GetDisplayNameForLanguageType(LanguageType language) {
274+
return SourceLanguage(language).GetDescription();
275+
}
276+
273277
void Language::PrintSupportedLanguagesForExpressions(Stream &s,
274278
llvm::StringRef prefix,
275279
llvm::StringRef suffix) {

0 commit comments

Comments
 (0)