[all-commits] [llvm/llvm-project] 9093b5: [lldb] Don't resolve dynamic type info if no named...

Nerixyz via All-commits all-commits at lists.llvm.org
Fri Aug 7 12:07:50 PDT 2026


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 9093b53d6162903bb9b050740132594fa29bd5ae
      https://github.com/llvm/llvm-project/commit/9093b53d6162903bb9b050740132594fa29bd5ae
  Author: Nerixyz <nerixdev at outlook.de>
  Date:   2026-08-07 (Fri, 07 Aug 2026)

  Changed paths:
    M lldb/source/Plugins/LanguageRuntime/CPlusPlus/CommonABIRuntime.cpp
    M lldb/source/Plugins/LanguageRuntime/CPlusPlus/CommonABIRuntime.h
    M lldb/source/Plugins/LanguageRuntime/CPlusPlus/ItaniumABIRuntime.cpp

  Log Message:
  -----------
  [lldb] Don't resolve dynamic type info if no named type exists (#214828)

Intends to fix the failure from
https://github.com/llvm/llvm-project/pull/212013#issuecomment-5220617065.

I overlooked that the original code returned an empty `TypeAndOrName` if
no type with the specified name was found:
https://github.com/llvm/llvm-project/blob/9b1218ce69e06a22699bc196888fc0396d989c81/lldb/source/Plugins/LanguageRuntime/CPlusPlus/ItaniumABIRuntime.cpp#L84-L88

In my PR, I always returned a `TypeAndOrName` with a name set. With this
change, we only set return a non-empty `TypeAndOrName` if there was any
type with the specified name.



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


More information about the All-commits mailing list