[Lldb-commits] [lldb] 8e0a087 - [lldb] Fix build failures introduced in f0731d5b61ba (NFCI)

Med Ismail Bennani via lldb-commits lldb-commits at lists.llvm.org
Tue Aug 29 08:35:07 PDT 2023


Author: Med Ismail Bennani
Date: 2023-08-29T16:34:56+01:00
New Revision: 8e0a087571a31057bb98939e3ada73227bed83c7

URL: https://github.com/llvm/llvm-project/commit/8e0a087571a31057bb98939e3ada73227bed83c7
DIFF: https://github.com/llvm/llvm-project/commit/8e0a087571a31057bb98939e3ada73227bed83c7.diff

LOG: [lldb] Fix build failures introduced in f0731d5b61ba (NFCI)

This patch should fix the build failures introduced by f0731d5b61ba.

This removes the use of the `STRING_EXTENSION_OUTSIDE` swig macro in SB
classes that don't implement a `GetDescription` method.

Signed-off-by: Med Ismail Bennani <ismail at bennani.ma>

Added: 
    

Modified: 
    lldb/bindings/interface/SBAttachInfoExtensions.i
    lldb/bindings/interface/SBBreakpointListExtensions.i
    lldb/bindings/interface/SBBroadcastExtensions.i
    lldb/bindings/interface/SBCommandInterpreterExtensions.i
    lldb/bindings/interface/SBCommandInterpreterRunOptionsExtensions.i
    lldb/bindings/interface/SBCommunicationExtensions.i
    lldb/bindings/interface/SBEnvironmentExtensions.i
    lldb/bindings/interface/SBExpressionOptionsExtensions.i
    lldb/bindings/interface/SBHostOSExtensions.i
    lldb/bindings/interface/SBLanguageRuntimeExtensions.i
    lldb/bindings/interface/SBLaunchInfoExtensions.i
    lldb/bindings/interface/SBListenerExtensions.i
    lldb/bindings/interface/SBMemoryRegionInfoListExtensions.i
    lldb/bindings/interface/SBPlatformConnectOptionsExtensions.i
    lldb/bindings/interface/SBPlatformExtensions.i
    lldb/bindings/interface/SBPlatformShellCommandExtensions.i
    lldb/bindings/interface/SBProcessInfoExtensions.i
    lldb/bindings/interface/SBQueueExtensions.i
    lldb/bindings/interface/SBQueueItemExtensions.i
    lldb/bindings/interface/SBReproducerExtensions.i
    lldb/bindings/interface/SBScriptObjectExtensions.i
    lldb/bindings/interface/SBSourceManagerExtensions.i
    lldb/bindings/interface/SBThreadCollectionExtensions.i
    lldb/bindings/interface/SBTraceCursorExtensions.i
    lldb/bindings/interface/SBTraceExtensions.i
    lldb/bindings/interface/SBTypeSummaryOptionsExtensions.i
    lldb/bindings/interface/SBVariablesOptionsExtensions.i

Removed: 
    


################################################################################
diff  --git a/lldb/bindings/interface/SBAttachInfoExtensions.i b/lldb/bindings/interface/SBAttachInfoExtensions.i
index 570c79e11e00cc..3057a038a91ebe 100644
--- a/lldb/bindings/interface/SBAttachInfoExtensions.i
+++ b/lldb/bindings/interface/SBAttachInfoExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBAttachInfo)
-
 %extend lldb::SBAttachInfo {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBBreakpointListExtensions.i b/lldb/bindings/interface/SBBreakpointListExtensions.i
index e376b07ba5fe9c..7036df002fd5d0 100644
--- a/lldb/bindings/interface/SBBreakpointListExtensions.i
+++ b/lldb/bindings/interface/SBBreakpointListExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBBreakpointList)
-
 %extend lldb::SBBreakpointList {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBBroadcastExtensions.i b/lldb/bindings/interface/SBBroadcastExtensions.i
index 6bcb84598fb46c..8baac6fc31bd16 100644
--- a/lldb/bindings/interface/SBBroadcastExtensions.i
+++ b/lldb/bindings/interface/SBBroadcastExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBBroadcaster)
-
 %extend lldb::SBBroadcaster {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBCommandInterpreterExtensions.i b/lldb/bindings/interface/SBCommandInterpreterExtensions.i
index 91d5b7bfefbd32..36d9c107d993ab 100644
--- a/lldb/bindings/interface/SBCommandInterpreterExtensions.i
+++ b/lldb/bindings/interface/SBCommandInterpreterExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBCommandInterpreter)
-
 %extend lldb::SBCommandInterpreter {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBCommandInterpreterRunOptionsExtensions.i b/lldb/bindings/interface/SBCommandInterpreterRunOptionsExtensions.i
index 358b2ba500c19b..8535e2ed4df327 100644
--- a/lldb/bindings/interface/SBCommandInterpreterRunOptionsExtensions.i
+++ b/lldb/bindings/interface/SBCommandInterpreterRunOptionsExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBCommandInterpreterRunOptions)
-
 %extend lldb::SBCommandInterpreterRunOptions {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBCommunicationExtensions.i b/lldb/bindings/interface/SBCommunicationExtensions.i
index 0e94f7f3e6407a..fbe80d33b736da 100644
--- a/lldb/bindings/interface/SBCommunicationExtensions.i
+++ b/lldb/bindings/interface/SBCommunicationExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBCommunication)
-
 %extend lldb::SBCommunication {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBEnvironmentExtensions.i b/lldb/bindings/interface/SBEnvironmentExtensions.i
index 170ffac1c0c357..f409b87b256be7 100644
--- a/lldb/bindings/interface/SBEnvironmentExtensions.i
+++ b/lldb/bindings/interface/SBEnvironmentExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBEnvironment)
-
 %extend lldb::SBEnvironment {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBExpressionOptionsExtensions.i b/lldb/bindings/interface/SBExpressionOptionsExtensions.i
index f70d8e8cef374e..cac385e5f5123a 100644
--- a/lldb/bindings/interface/SBExpressionOptionsExtensions.i
+++ b/lldb/bindings/interface/SBExpressionOptionsExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBExpressionOptions)
-
 %extend lldb::SBExpressionOptions {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBHostOSExtensions.i b/lldb/bindings/interface/SBHostOSExtensions.i
index 6a42a6f13906b8..eb2e124b492976 100644
--- a/lldb/bindings/interface/SBHostOSExtensions.i
+++ b/lldb/bindings/interface/SBHostOSExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBHostOS)
-
 %extend lldb::SBHostOS {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBLanguageRuntimeExtensions.i b/lldb/bindings/interface/SBLanguageRuntimeExtensions.i
index 82a9311d537d61..7d49f62c04bb86 100644
--- a/lldb/bindings/interface/SBLanguageRuntimeExtensions.i
+++ b/lldb/bindings/interface/SBLanguageRuntimeExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBLanguageRuntime)
-
 %extend lldb::SBLanguageRuntime {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBLaunchInfoExtensions.i b/lldb/bindings/interface/SBLaunchInfoExtensions.i
index 53a9a4d1b7aea3..fa7ddf2ddce582 100644
--- a/lldb/bindings/interface/SBLaunchInfoExtensions.i
+++ b/lldb/bindings/interface/SBLaunchInfoExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBLaunchInfo)
-
 %extend lldb::SBLaunchInfo {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBListenerExtensions.i b/lldb/bindings/interface/SBListenerExtensions.i
index 9c626d68056ac7..4464a1737da800 100644
--- a/lldb/bindings/interface/SBListenerExtensions.i
+++ b/lldb/bindings/interface/SBListenerExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBListener)
-
 %extend lldb::SBListener {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBMemoryRegionInfoListExtensions.i b/lldb/bindings/interface/SBMemoryRegionInfoListExtensions.i
index 09143e8e41a2dd..d2c0ebe4d07d78 100644
--- a/lldb/bindings/interface/SBMemoryRegionInfoListExtensions.i
+++ b/lldb/bindings/interface/SBMemoryRegionInfoListExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBMemoryRegionInfoList)
-
 %extend lldb::SBMemoryRegionInfoList {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBPlatformConnectOptionsExtensions.i b/lldb/bindings/interface/SBPlatformConnectOptionsExtensions.i
index 1aa6e7b5829064..cbc8ac0749e469 100644
--- a/lldb/bindings/interface/SBPlatformConnectOptionsExtensions.i
+++ b/lldb/bindings/interface/SBPlatformConnectOptionsExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBPlatformConnectOptions)
-
 %extend lldb::SBPlatformConnectOptions {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBPlatformExtensions.i b/lldb/bindings/interface/SBPlatformExtensions.i
index 29c7a8f1165bda..8328693b11797d 100644
--- a/lldb/bindings/interface/SBPlatformExtensions.i
+++ b/lldb/bindings/interface/SBPlatformExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBPlatform)
-
 %extend lldb::SBPlatform {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBPlatformShellCommandExtensions.i b/lldb/bindings/interface/SBPlatformShellCommandExtensions.i
index 56e8d62a7fc6ec..343f7933ea4628 100644
--- a/lldb/bindings/interface/SBPlatformShellCommandExtensions.i
+++ b/lldb/bindings/interface/SBPlatformShellCommandExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBPlatformShellCommand)
-
 %extend lldb::SBPlatformShellCommand {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBProcessInfoExtensions.i b/lldb/bindings/interface/SBProcessInfoExtensions.i
index 7307f370d115b2..2aab12b856885a 100644
--- a/lldb/bindings/interface/SBProcessInfoExtensions.i
+++ b/lldb/bindings/interface/SBProcessInfoExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBProcessInfo)
-
 %extend lldb::SBProcessInfo {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBQueueExtensions.i b/lldb/bindings/interface/SBQueueExtensions.i
index f03a2e9a078fd9..17804b9425ae59 100644
--- a/lldb/bindings/interface/SBQueueExtensions.i
+++ b/lldb/bindings/interface/SBQueueExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBQueue)
-
 %extend lldb::SBQueue {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBQueueItemExtensions.i b/lldb/bindings/interface/SBQueueItemExtensions.i
index 4faf05ed728442..fe9242283465b0 100644
--- a/lldb/bindings/interface/SBQueueItemExtensions.i
+++ b/lldb/bindings/interface/SBQueueItemExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBQueueItem)
-
 %extend lldb::SBQueueItem {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBReproducerExtensions.i b/lldb/bindings/interface/SBReproducerExtensions.i
index 93f52a77a693f0..fcce308df0caae 100644
--- a/lldb/bindings/interface/SBReproducerExtensions.i
+++ b/lldb/bindings/interface/SBReproducerExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBReproducer)
-
 %extend lldb::SBReproducer {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBScriptObjectExtensions.i b/lldb/bindings/interface/SBScriptObjectExtensions.i
index 99fc609ff9e0f6..35cbe641e90647 100644
--- a/lldb/bindings/interface/SBScriptObjectExtensions.i
+++ b/lldb/bindings/interface/SBScriptObjectExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBScriptObject)
-
 %extend lldb::SBScriptObject {
 #ifdef SWIGPYTHON
     %pythoncode %{

diff  --git a/lldb/bindings/interface/SBSourceManagerExtensions.i b/lldb/bindings/interface/SBSourceManagerExtensions.i
index 0e240cf0d4b682..903e777dde0f8e 100644
--- a/lldb/bindings/interface/SBSourceManagerExtensions.i
+++ b/lldb/bindings/interface/SBSourceManagerExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBSourceManager)
-
 %extend lldb::SBSourceManager {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBThreadCollectionExtensions.i b/lldb/bindings/interface/SBThreadCollectionExtensions.i
index 7b7e94ed9a991c..32982acfa9692b 100644
--- a/lldb/bindings/interface/SBThreadCollectionExtensions.i
+++ b/lldb/bindings/interface/SBThreadCollectionExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBThreadCollection)
-
 %extend lldb::SBThreadCollection {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBTraceCursorExtensions.i b/lldb/bindings/interface/SBTraceCursorExtensions.i
index ce1f1d1bc34152..9ccff9b1b5a2c2 100644
--- a/lldb/bindings/interface/SBTraceCursorExtensions.i
+++ b/lldb/bindings/interface/SBTraceCursorExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBTraceCursor)
-
 %extend lldb::SBTraceCursor {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBTraceExtensions.i b/lldb/bindings/interface/SBTraceExtensions.i
index ed26ccc6849bfd..dc169600cca6ae 100644
--- a/lldb/bindings/interface/SBTraceExtensions.i
+++ b/lldb/bindings/interface/SBTraceExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBTrace)
-
 %extend lldb::SBTrace {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBTypeSummaryOptionsExtensions.i b/lldb/bindings/interface/SBTypeSummaryOptionsExtensions.i
index e7a41a853840e9..bd59604629b6d0 100644
--- a/lldb/bindings/interface/SBTypeSummaryOptionsExtensions.i
+++ b/lldb/bindings/interface/SBTypeSummaryOptionsExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBTypeSummaryOptions)
-
 %extend lldb::SBTypeSummaryOptions {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject

diff  --git a/lldb/bindings/interface/SBVariablesOptionsExtensions.i b/lldb/bindings/interface/SBVariablesOptionsExtensions.i
index a9b79a64b27543..cec92e79b2add3 100644
--- a/lldb/bindings/interface/SBVariablesOptionsExtensions.i
+++ b/lldb/bindings/interface/SBVariablesOptionsExtensions.i
@@ -1,5 +1,3 @@
-STRING_EXTENSION_OUTSIDE(SBVariablesOptions)
-
 %extend lldb::SBVariablesOptions {
 #ifdef SWIGPYTHON
     // operator== is a free function, which swig does not handle, so we inject


        


More information about the lldb-commits mailing list