summaryrefslogtreecommitdiff
path: root/lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedProcessPythonInterface.cpp
diff options
context:
space:
mode:
authorDavid Spickett <david.spickett@linaro.org>2024-01-26 13:52:04 +0000
committerDavid Spickett <david.spickett@linaro.org>2024-01-26 14:06:49 +0000
commitbb945fcd4a54c2c8f898e2bdc0d65fae841a1909 (patch)
treeb6881d45878c61af91b6ac755a9e63f8038bab80 /lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedProcessPythonInterface.cpp
parent1b7b40bf5d6847623e2737d4fe7b31a434195852 (diff)
Reland "[lldb][ObjC][NFC] Fix c++20 gcc compile errors"
clang's -Wdtor name is correct, but the standard may have not intended that meaning, according to https://bugs.llvm.org/show_bug.cgi?id=46979#c1. Some of the wording may have changed in 20/23, but we of course need to support c++17 as well as that's our default. One workaround would be to explicitly open the namespaces, then declare the destructor inside that. Another as shown in the bug report is to repeat the class name, without the template arguments, before the ::~. For example `Bar::Foo<T>::Foo::~Foo`. (this extra Foo is the injected class name https://en.cppreference.com/w/cpp/language/injected-class-name) I chose to do this because it's the smallest change. It works with gcc-13 and clang in c++17 and 20 modes (https://godbolt.org/z/fqs4fGE7T).
Diffstat (limited to 'lldb/source/Plugins/ScriptInterpreter/Python/Interfaces/ScriptedProcessPythonInterface.cpp')
0 files changed, 0 insertions, 0 deletions