[all-commits] [llvm/llvm-project] 4b8970: [LLDB][NativePDB] Consolidate simple types (#163209)

nerix via All-commits all-commits at lists.llvm.org
Wed Oct 15 04:42:36 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 4b89704504dde687ba7983e034cb246581fd1407
      https://github.com/llvm/llvm-project/commit/4b89704504dde687ba7983e034cb246581fd1407
  Author: nerix <nerixdev at outlook.de>
  Date:   2025-10-15 (Wed, 15 Oct 2025)

  Changed paths:
    M lldb/source/Plugins/SymbolFile/NativePDB/PdbUtil.cpp
    M lldb/source/Plugins/SymbolFile/NativePDB/SymbolFileNativePDB.cpp
    M lldb/test/Shell/SymbolFile/NativePDB/local-variables-registers.s
    M lldb/test/Shell/SymbolFile/NativePDB/simple-types.cpp

  Log Message:
  -----------
  [LLDB][NativePDB] Consolidate simple types (#163209)

This aligns the simple types created by the native plugin with the ones
from DIA as well as LLVM and the original cvdump.

- A few type names weren't handled when creating the LLDB `Type` name
(e.g. `short`)
- 64-bit integers were created as `(u)int64_t` and are now created as
`(unsigned) long long` (matches DIA)
- 128-bit integers (only supported by clang-cl) weren't created as types
(they have `SimpleTypeKind::(U)Int128Oct`)
- All complex types had the same name - now they have `_Complex
<float-type>`

Some types like `SimpleTypeKind::Float48` can't be tested because they
can't be created in C++.



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