Author: Michael Buch
Date: 2025-04-26T01:00:17+01:00
New Revision: 5137587fb1dc878e2dca31c4b929a3ca99f70ace

URL: 
https://github.com/llvm/llvm-project/commit/5137587fb1dc878e2dca31c4b929a3ca99f70ace
DIFF: 
https://github.com/llvm/llvm-project/commit/5137587fb1dc878e2dca31c4b929a3ca99f70ace.diff

LOG: [lldb][test] MangledTest: don't use designated initializers

It's a C++20 feature and is causing some buildbots to fail:
```
FAILED: 
tools/lldb/unittests/Core/CMakeFiles/LLDBCoreTests.dir/MangledTest.cpp.obj
ccache 
C:\PROGRA~1\MICROS~1\2022\COMMUN~1\VC\Tools\MSVC\1443~1.348\bin\Hostx64\x64\cl.exe
  /nologo /TP -DGTEST_HAS_RTTI=0 -DLLVM_BUILD_STATIC -DUNICODE 
-D_CRT_NONSTDC_NO_DEPRECATE -D_CRT_NONSTDC_NO_WARNINGS 
-D_CRT_SECURE_NO_DEPRECATE -D_CRT_SECURE_NO_WARNINGS 
-D_ENABLE_EXTENDED_ALIGNED_STORAGE -D_GLIBCXX_ASSERTIONS -D_HAS_EXCEPTIONS=0 
-D_SCL_SECURE_NO_DEPRECATE -D_SCL_SECURE_NO_WARNINGS -D_UNICODE 
-D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS 
-IC:\buildbot\as-builder-10\lldb-x-aarch64\build\tools\lldb\unittests\Core 
-IC:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\lldb\unittests\Core 
-IC:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\lldb\include 
-IC:\buildbot\as-builder-10\lldb-x-aarch64\build\tools\lldb\include 
-IC:\buildbot\as-builder-10\lldb-x-aarch64\build\include 
-IC:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\llvm\include 
-IC:\Python312\include 
-IC:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\llvm\..\clang\include 
-IC:\buildbot\as-builder-10\lldb-x-aarch64\build\tools\lldb\..\clang\include 
-IC:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\lldb\source 
-IC:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\lldb\unittests 
-IC:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\third-party\unittest\googletest\include
 
-IC:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\third-party\unittest\googlemock\include
 -D__OPTIMIZE__ /Zc:inline /Zc:preprocessor /Zc:__cplusplus /Oi /bigobj 
/permissive- /W4 -wd4141 -wd4146 -wd4244 -wd4267 -wd4291 -wd4351 -wd4456 
-wd4457 -wd4458 -wd4459 -wd4503 -wd4624 -wd4722 -wd4100 -wd4127 -wd4512 -wd4505 
-wd4610 -wd4510 -wd4702 -wd4245 -wd4706 -wd4310 -wd4701 -wd4703 -wd4389 -wd4611 
-wd4805 -wd4204 -wd4577 -wd4091 -wd4592 -wd4319 -wd4709 -wd5105 -wd4324 -wd4251 
-wd4275 -w14062 -we4238 /Gw /O2 /Ob2  -MD   -wd4018 -wd4068 -wd4150 -wd4201 
-wd4251 -wd4521 -wd4530 -wd4589  /EHs-c- /GR- -UNDEBUG -std:c++17 /showIncludes 
/Fotools\lldb\unittests\Core\CMakeFiles\LLDBCoreTests.dir\MangledTest.cpp.obj 
/Fdtools\lldb\unittests\Core\CMakeFiles\LLDBCoreTests.dir\ /FS -c 
C:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\lldb\unittests\Core\MangledTest.cpp
C:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\lldb\unittests\Core\MangledTest.cpp(416):
 error C7555: use of designated initializers requires at least '/std:c++20'
C:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\lldb\unittests\Core\MangledTest.cpp(415):
 error C7556: cannot mix designated-initializers with 
non-designated-initializers
C:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\lldb\unittests\Core\MangledTest.cpp(418):
 error C7555: use of designated initializers requires at least '/std:c++20'
C:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\lldb\unittests\Core\MangledTest.cpp(423):
 error C7555: use of designated initializers requires at least '/std:c++20'
C:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\lldb\unittests\Core\MangledTest.cpp(422):
 error C7556: cannot mix designated-initializers with 
non-designated-initializers
C:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\lldb\unittests\Core\MangledTest.cpp(424):
 error C7555: use of designated initializers requires at least '/std:c++20'
C:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\lldb\unittests\Core\MangledTest.cpp(429):
 error C7555: use of designated initializers requires at least '/std:c++20'
C:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\lldb\unittests\Core\MangledTest.cpp(428):
 error C7556: cannot mix designated-initializers with 
non-designated-initializers
C:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\lldb\unittests\Core\MangledTest.cpp(431):
 error C7555: use of designated initializers requires at least '/std:c++20'
C:\buildbot\as-builder-10\lldb-x-aarch64\llvm-project\lldb\unittests\Core\MangledTest.cpp(436):
 error C7555: use of designated initializers requires at least '/std:c++20'

```

Added: 
    

Modified: 
    lldb/unittests/Core/MangledTest.cpp

Removed: 
    


################################################################################
diff  --git a/lldb/unittests/Core/MangledTest.cpp 
b/lldb/unittests/Core/MangledTest.cpp
index 7dfd4b2a81f9b..8a452a84614ef 100644
--- a/lldb/unittests/Core/MangledTest.cpp
+++ b/lldb/unittests/Core/MangledTest.cpp
@@ -413,115 +413,115 @@ struct DemanglingPartsTestCase {
 DemanglingPartsTestCase g_demangling_parts_test_cases[] = {
     // clang-format off
    { 
"_ZNVKO3BarIN2ns3QuxIiEEE1CIPFi3FooIS_IiES6_EEE6methodIS6_EENS5_IT_SC_E5InnerIiEESD_SD_",
-     { .BasenameRange = {92, 98}, .ScopeRange = {36, 92}, .ArgumentsRange = { 
108, 158 },
-       .QualifiersRange = {158, 176} },
-     .basename = "method",
-     .scope = "Bar<ns::Qux<int>>::C<int (*)(Foo<Bar<int>, Bar<int>>)>::",
-     .qualifiers = " const volatile &&"
+     { /*.BasenameRange=*/{92, 98}, /*.ScopeRange=*/{36, 92}, 
/*.ArgumentsRange=*/{ 108, 158 },
+       /*.QualifiersRange=*/{158, 176} },
+     /*.basename=*/"method",
+     /*.scope=*/"Bar<ns::Qux<int>>::C<int (*)(Foo<Bar<int>, Bar<int>>)>::",
+     /*.qualifiers=*/" const volatile &&"
    },
    { "_Z7getFuncIfEPFiiiET_",
-     { .BasenameRange = {6, 13}, .ScopeRange = {6, 6}, .ArgumentsRange = { 20, 
27 }, .QualifiersRange = {38, 38} },
-     .basename = "getFunc",
-     .scope = "",
-     .qualifiers = ""
+     { /*.BasenameRange=*/{6, 13}, /*.ScopeRange=*/{6, 6}, 
/*.ArgumentsRange=*/{ 20, 27 }, /*.QualifiersRange=*/{38, 38} },
+     /*.basename=*/"getFunc",
+     /*.scope=*/"",
+     /*.qualifiers=*/""
    },
    { "_ZN1f1b1c1gEv",
-     { .BasenameRange = {9, 10}, .ScopeRange = {0, 9}, .ArgumentsRange = { 10, 
12 },
-       .QualifiersRange = {12, 12} },
-     .basename = "g",
-     .scope = "f::b::c::",
-     .qualifiers = ""
+     { /*.BasenameRange=*/{9, 10}, /*.ScopeRange=*/{0, 9}, 
/*.ArgumentsRange=*/{ 10, 12 },
+       /*.QualifiersRange=*/{12, 12} },
+     /*.basename=*/"g",
+     /*.scope=*/"f::b::c::",
+     /*.qualifiers=*/""
    },
    { "_ZN5test73fD1IiEEDTcmtlNS_1DEL_ZNS_1bEEEcvT__EES2_",
-     { .BasenameRange = {45, 48}, .ScopeRange = {38, 45}, .ArgumentsRange = { 
53, 58 },
-       .QualifiersRange = {58, 58} },
-     .basename = "fD1",
-     .scope = "test7::",
-     .qualifiers = ""
+     { /*.BasenameRange=*/{45, 48}, /*.ScopeRange=*/{38, 45}, 
/*.ArgumentsRange=*/{ 53, 58 },
+       /*.QualifiersRange=*/{58, 58} },
+     /*.basename=*/"fD1",
+     /*.scope=*/"test7::",
+     /*.qualifiers=*/""
    },
    { "_ZN5test73fD1IiEEDTcmtlNS_1DEL_ZNS_1bINDT1cE1dEEEEEcvT__EES2_",
-     { .BasenameRange = {61, 64}, .ScopeRange = {54, 61}, .ArgumentsRange = { 
69, 79 },
-       .QualifiersRange = {79, 79} },
-     .basename = "fD1",
-     .scope = "test7::",
-     .qualifiers = ""
+     { /*.BasenameRange=*/{61, 64}, /*.ScopeRange=*/{54, 61}, 
/*.ArgumentsRange=*/{ 69, 79 },
+       /*.QualifiersRange=*/{79, 79} },
+     /*.basename=*/"fD1",
+     /*.scope=*/"test7::",
+     /*.qualifiers=*/""
    },
    { 
"_ZN5test7INDT1cE1dINDT1cE1dEEEE3fD1INDT1cE1dINDT1cE1dEEEEEDTcmtlNS_1DEL_ZNS_1bINDT1cE1dEEEEEcvT__EES2_",
-     { .BasenameRange = {120, 123}, .ScopeRange = {81, 120}, .ArgumentsRange = 
{ 155, 168 },
-       .QualifiersRange = {168, 168} },
-     .basename = "fD1",
-     .scope = "test7<decltype(c)::d<decltype(c)::d>>::",
-     .qualifiers = ""
+     { /*.BasenameRange=*/{120, 123}, /*.ScopeRange=*/{81, 120}, 
/*.ArgumentsRange=*/{ 155, 168 },
+       /*.QualifiersRange=*/{168, 168} },
+     /*.basename=*/"fD1",
+     /*.scope=*/"test7<decltype(c)::d<decltype(c)::d>>::",
+     /*.qualifiers=*/""
    },
    { 
"_ZN8nlohmann16json_abi_v3_11_310basic_jsonINSt3__13mapENS2_6vectorENS2_12basic_stringIcNS2_11char_traitsIcEENS2_9allocatorIcEEEEbxydS8_NS0_14adl_serializerENS4_IhNS8_IhEEEEvE5parseIRA29_KcEESE_OT_NS2_8functionIFbiNS0_6detail13parse_event_tERSE_EEEbb",
-     { .BasenameRange = {687, 692}, .ScopeRange = {343, 687}, .ArgumentsRange 
= { 713, 1174 },
-       .QualifiersRange = {1174, 1174} },
-     .basename = "parse",
-     .scope = "nlohmann::json_abi_v3_11_3::basic_json<std::__1::map, 
std::__1::vector, std::__1::basic_string<char, std::__1::char_traits<char>, 
std::__1::allocator<char>>, bool, long long, unsigned long long, double, 
std::__1::allocator, nlohmann::json_abi_v3_11_3::adl_serializer, 
std::__1::vector<unsigned char, std::__1::allocator<unsigned char>>, void>::",
-     .qualifiers = ""
+     { /*.BasenameRange=*/{687, 692}, /*.ScopeRange=*/{343, 687}, 
/*.ArgumentsRange=*/{ 713, 1174 },
+       /*.QualifiersRange=*/{1174, 1174} },
+     /*.basename=*/"parse",
+     /*.scope=*/"nlohmann::json_abi_v3_11_3::basic_json<std::__1::map, 
std::__1::vector, std::__1::basic_string<char, std::__1::char_traits<char>, 
std::__1::allocator<char>>, bool, long long, unsigned long long, double, 
std::__1::allocator, nlohmann::json_abi_v3_11_3::adl_serializer, 
std::__1::vector<unsigned char, std::__1::allocator<unsigned char>>, void>::",
+     /*.qualifiers=*/""
    },
    { 
"_ZN8nlohmann16json_abi_v3_11_310basic_jsonINSt3__13mapENS2_6vectorENS2_12basic_stringIcNS2_11char_traitsIcEENS2_9allocatorIcEEEEbxydS8_NS0_14adl_serializerENS4_IhNS8_IhEEEEvEC1EDn",
-     { .BasenameRange = {344, 354}, .ScopeRange = {0, 344}, .ArgumentsRange = 
{ 354, 370 },
-       .QualifiersRange = {370, 370} },
-     .basename = "basic_json",
-     .scope = "nlohmann::json_abi_v3_11_3::basic_json<std::__1::map, 
std::__1::vector, std::__1::basic_string<char, std::__1::char_traits<char>, 
std::__1::allocator<char>>, bool, long long, unsigned long long, double, 
std::__1::allocator, nlohmann::json_abi_v3_11_3::adl_serializer, 
std::__1::vector<unsigned char, std::__1::allocator<unsigned char>>, void>::",
-     .qualifiers = ""
+     { /*.BasenameRange=*/{344, 354}, /*.ScopeRange=*/{0, 344}, 
/*.ArgumentsRange=*/{ 354, 370 },
+       /*.QualifiersRange=*/{370, 370} },
+     /*.basename=*/"basic_json",
+     /*.scope=*/"nlohmann::json_abi_v3_11_3::basic_json<std::__1::map, 
std::__1::vector, std::__1::basic_string<char, std::__1::char_traits<char>, 
std::__1::allocator<char>>, bool, long long, unsigned long long, double, 
std::__1::allocator, nlohmann::json_abi_v3_11_3::adl_serializer, 
std::__1::vector<unsigned char, std::__1::allocator<unsigned char>>, void>::",
+     /*.qualifiers=*/""
    },
    { "_Z3fppIiEPFPFvvEiEf",
-     { .BasenameRange = {10, 13}, .ScopeRange = {10, 10}, .ArgumentsRange = { 
18, 25 }, .QualifiersRange = {34,34} },
-     .basename = "fpp",
-     .scope = "",
-     .qualifiers = ""
+     { /*.BasenameRange=*/{10, 13}, /*.ScopeRange=*/{10, 10}, 
/*.ArgumentsRange=*/{ 18, 25 }, /*.QualifiersRange=*/{34,34} },
+     /*.basename=*/"fpp",
+     /*.scope=*/"",
+     /*.qualifiers=*/""
    },
    { "_Z3fppIiEPFPFvvEN2ns3FooIiEEEf",
-     { .BasenameRange = {10, 13}, .ScopeRange = {10, 10}, .ArgumentsRange = { 
18, 25 },
-       .QualifiersRange = {43, 43} },
-     .basename = "fpp",
-     .scope = "",
-     .qualifiers = ""
+     { /*.BasenameRange=*/{10, 13}, /*.ScopeRange=*/{10, 10}, 
/*.ArgumentsRange=*/{ 18, 25 },
+       /*.QualifiersRange=*/{43, 43} },
+     /*.basename=*/"fpp",
+     /*.scope=*/"",
+     /*.qualifiers=*/""
    },
    { "_Z3fppIiEPFPFvPFN2ns3FooIiEENS2_3BarIfE3QuxEEEPFS2_S2_EEf",
-     { .BasenameRange = {10, 13}, .ScopeRange = {10, 10}, .ArgumentsRange = { 
18, 25 },
-       .QualifiersRange = {108, 108} },
-     .basename = "fpp",
-     .scope = "",
-     .qualifiers = ""
+     { /*.BasenameRange=*/{10, 13}, /*.ScopeRange=*/{10, 10}, 
/*.ArgumentsRange=*/{ 18, 25 },
+       /*.QualifiersRange=*/{108, 108} },
+     /*.basename=*/"fpp",
+     /*.scope=*/"",
+     /*.qualifiers=*/""
    },
    { "_ZN2ns8HasFuncsINS_3FooINS1_IiE3BarIfE3QuxEEEE3fppIiEEPFPFvvEiEf",
-     { .BasenameRange = {64, 67}, .ScopeRange = {10, 64}, .ArgumentsRange = { 
72, 79 },
-       .QualifiersRange = {88, 88} },
-     .basename = "fpp",
-     .scope = "ns::HasFuncs<ns::Foo<ns::Foo<int>::Bar<float>::Qux>>::",
-     .qualifiers = ""
+     { /*.BasenameRange=*/{64, 67}, /*.ScopeRange=*/{10, 64}, 
/*.ArgumentsRange=*/{ 72, 79 },
+       /*.QualifiersRange=*/{88, 88} },
+     /*.basename=*/"fpp",
+     /*.scope=*/"ns::HasFuncs<ns::Foo<ns::Foo<int>::Bar<float>::Qux>>::",
+     /*.qualifiers=*/""
    },
    { "_ZN2ns8HasFuncsINS_3FooINS1_IiE3BarIfE3QuxEEEE3fppIiEEPFPFvvES2_Ef",
-     { .BasenameRange = {64, 67}, .ScopeRange = {10, 64}, .ArgumentsRange = { 
72, 79 },
-       .QualifiersRange = {97, 97} },
-     .basename = "fpp",
-     .scope = "ns::HasFuncs<ns::Foo<ns::Foo<int>::Bar<float>::Qux>>::",
-     .qualifiers = "",
+     { /*.BasenameRange=*/{64, 67}, /*.ScopeRange=*/{10, 64}, 
/*.ArgumentsRange=*/{ 72, 79 },
+       /*.QualifiersRange=*/{97, 97} },
+     /*.basename=*/"fpp",
+     /*.scope=*/"ns::HasFuncs<ns::Foo<ns::Foo<int>::Bar<float>::Qux>>::",
+     /*.qualifiers=*/"",
    },
    { 
"_ZN2ns8HasFuncsINS_3FooINS1_IiE3BarIfE3QuxEEEE3fppIiEEPFPFvPFS2_S5_EEPFS2_S2_EEf",
-     { .BasenameRange = {64, 67}, .ScopeRange = {10, 64}, .ArgumentsRange = { 
72, 79 },
-       .QualifiersRange = {162, 162} },
-     .basename = "fpp",
-     .scope = "ns::HasFuncs<ns::Foo<ns::Foo<int>::Bar<float>::Qux>>::",
-     .qualifiers = "",
+     { /*.BasenameRange=*/{64, 67}, /*.ScopeRange=*/{10, 64}, 
/*.ArgumentsRange=*/{ 72, 79 },
+       /*.QualifiersRange=*/{162, 162} },
+     /*.basename=*/"fpp",
+     /*.scope=*/"ns::HasFuncs<ns::Foo<ns::Foo<int>::Bar<float>::Qux>>::",
+     /*.qualifiers=*/"",
    },
    { "_ZNKO2ns3ns23Bar3fooIiEEPFPFNS0_3FooIiEEiENS3_IfEEEi",
-     { .BasenameRange = {37, 40}, .ScopeRange = {23, 37}, .ArgumentsRange = { 
45, 50 },
-       .QualifiersRange = {78, 87} },
-     .basename = "foo",
-     .scope = "ns::ns2::Bar::",
-     .qualifiers = " const &&",
+     { /*.BasenameRange=*/{37, 40}, /*.ScopeRange=*/{23, 37}, 
/*.ArgumentsRange=*/{ 45, 50 },
+       /*.QualifiersRange=*/{78, 87} },
+     /*.basename=*/"foo",
+     /*.scope=*/"ns::ns2::Bar::",
+     /*.qualifiers=*/" const &&",
    },
    { "_ZTV11ImageLoader",
-     { .BasenameRange = {0, 0}, .ScopeRange = {0, 0}, .ArgumentsRange = { 0, 0 
},
-       .QualifiersRange = {0, 0} },
-     .basename = "",
-     .scope = "",
-     .qualifiers = "",
-     .valid_basename = false
+     { /*.BasenameRange=*/{0, 0}, /*.ScopeRange=*/{0, 0}, 
/*.ArgumentsRange=*/{ 0, 0 },
+       /*.QualifiersRange=*/{0, 0} },
+     /*.basename=*/"",
+     /*.scope=*/"",
+     /*.qualifiers=*/"",
+     /*.valid_basename=*/false
    }
     // clang-format on
 };


        
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to