diff options
author | Aaron Smith <aaron.smith@microsoft.com> | 2018-01-23 20:35:19 +0000 |
---|---|---|
committer | Aaron Smith <aaron.smith@microsoft.com> | 2018-01-23 20:35:19 +0000 |
commit | ec40f818c630156db9318e0aa8206acb334b9ac8 (patch) | |
tree | 235657b8c1f2984f010f41a660fe01a9c0d4bbdd /clang/lib/AST/MicrosoftMangle.cpp | |
parent | a9ae2fe782c5d80d1c4f8443a13bf12b33add32f (diff) | |
download | bcm5719-llvm-ec40f818c630156db9318e0aa8206acb334b9ac8.tar.gz bcm5719-llvm-ec40f818c630156db9318e0aa8206acb334b9ac8.zip |
[SymbolFilePDB] Fix null array access when parsing the type of a function without any arguments, i.e. 'int main()' and add support to test it
Summary:
- Fix a null array access bug. This happens when creating the lldb type for a function that has no argument.
- Implement SymbolFilePDB::ParseTypes method. Using `lldb-test symbols` will show all supported types in the target.
- Create lldb types for variadic function, PDBSymbolTypePointer, PDBSymbolTypeBuiltin
- The underlying builtin type for PDBSymbolTypeEnum is always `Int`, correct it with the very first enumerator's encoding if any. This is more accurate when the underlying type is not signed or another integer type.
- Fix a bug when the compiler type is not created based on PDB_BuiltinType. For example, basic type `long` is of same width as `int` in a 32-bit target, and the compiler type of former one will be represented by the one generated for latter if using the default method. Introduce a static function GetBuiltinTypeForPDBEncodingAndBitSize to correct this issue.
- Basic type `long double` and `double` have the same bit size in MSVC and there is no information in a PDB to distinguish them. The compiler type of the former one is represented by the latter's.
- There is no line information about typedef, enum etc in a PDB and the source and line information for them are not shown.
- There is no information about scoped enumeration. The compiler type is represented as an unscoped one.
Reviewers: zturner, lldb-commits
Reviewed By: zturner
Subscribers: majnemer, llvm-commits
Differential Revision: https://reviews.llvm.org/D42434
llvm-svn: 323255
Diffstat (limited to 'clang/lib/AST/MicrosoftMangle.cpp')
0 files changed, 0 insertions, 0 deletions