llvm-project.git/lldb/test/Shell/SymbolFile/DWARF/x86/find-variable-file.cpp, branch main Unnamed repository; edit this file 'description' to name the repository. [NFCI][lldb][test] Fix mismatched C/C++ substitutions (#165773) 2025-10-30T21:18:32+00:00 Raul Tambre raul@tambre.ee 2025-10-30T21:18:32+00:00 21041c92925faf529717d96279b6d8f20b17c70c Most of the cases were where a C++ file was being compiled with the C substitution. There were a few cases of the opposite though. LLDB seems to be the only real culprit in the LLVM codebase for these mismatches. Rest of the LLVM presumably sticks at least language-specific options in the common substitutions making the mistakes immediately apparent. I found these by using Clang frontend configuration files containing language-specific options for both C and C++ (e.g. `-std=c2y` and `-std=c++26`). 
Most of the cases were where a C++ file was being compiled with the C substitution.
There were a few cases of the opposite though.

LLDB seems to be the only real culprit in the LLVM codebase for these mismatches.
Rest of the LLVM presumably sticks at least language-specific options in the common substitutions
making the mistakes immediately apparent.

I found these by using Clang frontend configuration files containing language-specific options for
both C and C++ (e.g. `-std=c2y` and `-std=c++26`).
 [nfc] [lldb] Assertions for D106270 - [DWARF5] Fix offset check when using .debug_names 2021-08-10T18:43:24+00:00 Jan Kratochvil jan.kratochvil@redhat.com 2021-08-10T18:39:40+00:00 f3932b9a0b0b7787ccd3572bad134acc4146acaa Skeleton vs. DWO units mismatch has been fixed in D106270. As they both have type DWARFUnit it is a bit difficult to debug. So it is better to make it safe against future changes. Reviewed By: kimanh, clayborg Differential Revision: https://reviews.llvm.org/D107659 
Skeleton vs. DWO units mismatch has been fixed in D106270. As they both
have type DWARFUnit it is a bit difficult to debug. So it is better to
make it safe against future changes.

Reviewed By: kimanh, clayborg

Differential Revision: https://reviews.llvm.org/D107659
[DWARF5] Fix offset check when using .debug_names 2021-08-09T11:15:14+00:00 Kim-Anh Tran kimanh@chromium.org 2021-08-09T11:14:41+00:00 0dda5425318a5785e3e19cfe369a43b221b9642d When going through the CU entries in the name index, make sure to compare the name entry's CU offset against the skeleton CU's offset. Previously there would be a mismatch, since the wrong offset was compared, and thus no suitable entry was found. Reviewed By: jankratochvil Differential Revision: https://reviews.llvm.org/D106270 
When going through the CU entries in the name index,
make sure to compare the name entry's CU
offset against the skeleton CU's offset.

Previously there would be a mismatch, since the
wrong offset was compared, and thus no suitable
entry was found.

Reviewed By: jankratochvil

Differential Revision: https://reviews.llvm.org/D106270
[lldb] Require x86 backend for a bunch of DWARF tests 2021-04-13T08:47:41+00:00 David Spickett david.spickett@linaro.org 2021-04-09T13:14:31+00:00 8a64d80a959bf2844df33f9112e456f33de7b468 By moving them into a folder with a local lit config requiring x86. All these tests use x86 target triples. There are two tests that require target-x86_64 because they run program files (instead of just needing the backend). Those are moved to the x86 folder also but their REQUIRES are unchanged. Reviewed By: JDevlieghere Differential Revision: https://reviews.llvm.org/D100193 
By moving them into a folder with a local lit config
requiring x86. All these tests use x86 target triples.

There are two tests that require target-x86_64 because
they run program files (instead of just needing the backend).
Those are moved to the x86 folder also but their REQUIRES are
unchanged.

Reviewed By: JDevlieghere

Differential Revision: https://reviews.llvm.org/D100193