Changes in TestCSizeof.py

2238dcc39358353cac21df75c3c3286ab20b8f53 - [NFC][Py Reformat] Reformat python files in lldb

Thu, 25 May 2023 15:48:57 +0000

[NFC][Py Reformat] Reformat python files in lldbThis is an ongoing series of commits that are reformatting our Pythoncode. Reformatting is done with `black` (23.1.0).If you end up having problems merging this commit because you have madechanges to a python file, the best way to handle that is to run `gitcheckout --ours ` and then reformat it with black.RFC: https://discourse.llvm.org/t/rfc-document-and-standardize-python-code-styleDifferential revision: https://reviews.llvm.org/D151460 List of files: /llvm-project/lldb/test/API/lang/c/sizeof/TestCSizeof.py

4cc8f2a017c76af25234afc7c380550e9c93135c - [lldb][tests] Automatically call compute_mydir (NFC)

Fri, 17 Jun 2022 00:38:47 +0000

[lldb][tests] Automatically call compute_mydir (NFC)Eliminate boilerplate of having each test manually assign to `mydir` by calling`compute_mydir` in lldbtest.py.Differential Revision: https://reviews.llvm.org/D128077 List of files: /llvm-project/lldb/test/API/lang/c/sizeof/TestCSizeof.py

2ce889fa4e5cab75fc65d03a4dfae52784d57db9 - [lldb][NFC] Add size tests for empty records with alignment and with empty members

Mon, 30 Aug 2021 13:16:18 +0000

[lldb][NFC] Add size tests for empty records with alignment and with empty membersThis came up during the Windows bot failure discussing after D105471 . Seealso 3d9a9fa6911a5228ce799a7c639e94d322678934 . List of files: /llvm-project/lldb/test/API/lang/c/sizeof/TestCSizeof.py

67c588c481bb2fb9f72459fbc205c3e1ee8c4b3b - [lldb] Generalize empty record size computation to avoid giving empty C++ structs a size of 0

Thu, 22 Jul 2021 11:25:16 +0000

[lldb] Generalize empty record size computation to avoid giving empty C++ structs a size of 0C doesn't allow empty structs but Clang/GCC support them and give them a size of 0.LLDB implements this by checking the tag kind and if it's `DW_TAG_structure_type` thenwe give it a size of 0 via an empty external RecordLayout. This is done because ourinternal TypeSystem is always in C++ mode (which means we would give them a sizeof 1).The current check for when we have this special case is currently too lax as types with`DW_TAG_structure_type` can also occur in C++ with types defined using the `struct`keyword. This means that in a C++ program with `struct Empty{};`, LLDB would return`0` for `sizeof(Empty)` even though the correct size is 1.This patch removes this special case and replaces it with a generic approach that justassigns empty structs the byte_size as specified in DWARF. The GCC/Clang specialcase is handles as they both emit an explicit `DW_AT_byte_size` of 0. And if anothercompiler decides to use a different byte size for this case then this should also behandled by the same code as long as that information is provided via `DW_AT_byte_size`.Reviewed By: werat, shafikDifferential Revision: https://reviews.llvm.org/D105471 List of files: /llvm-project/lldb/test/API/lang/c/sizeof/TestCSizeof.py