Author: Daniel Sanders
Date: 2025-10-27T10:25:58-07:00
New Revision: e903494967ad9b9a68c62f984cf53db24b4532e9

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

LOG: [lldb] Fix TestVTableValue.py test_overwrite_vtable test (#164910)

Some machines have read-only vtables but this test expects to overwrite
them. Use -no_data_const to ensure the vtable is writable

Added: 
    

Modified: 
    lldb/test/API/functionalities/vtable/Makefile

Removed: 
    


################################################################################
diff  --git a/lldb/test/API/functionalities/vtable/Makefile 
b/lldb/test/API/functionalities/vtable/Makefile
index 99998b20bcb05..cbd7d472fb768 100644
--- a/lldb/test/API/functionalities/vtable/Makefile
+++ b/lldb/test/API/functionalities/vtable/Makefile
@@ -1,3 +1,9 @@
 CXX_SOURCES := main.cpp
 
+ifeq "$(OS)" "Darwin"
+    # Make vtables writable for test_overwrite_vtable test
+    # The -no_data_const flag prevents vtables from being placed in 
__DATA_CONST
+    LD_EXTRAS := -Wl,-no_data_const
+endif
+
 include Makefile.rules


        
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to