[Lldb-commits] [lldb] b95c9b7 - [lldb] Remove the last radar link in LLDB source code (NFC)

Jonas Devlieghere via lldb-commits lldb-commits at lists.llvm.org
Sun Jul 30 15:29:40 PDT 2023


Author: Jonas Devlieghere
Date: 2023-07-30T15:28:07-07:00
New Revision: b95c9b7af315510306dfdd82e49b3dbac407e142

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

LOG: [lldb] Remove the last radar link in LLDB source code (NFC)

This was probably the only really useful radar in our source code. It's
a request to be able to tell if __LINKEDIT has been mapped or not. I've
left a comment in the radar that we should update the corresponding code
if and when such an ability becomes available.

Added: 
    

Modified: 
    lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.cpp

Removed: 
    


################################################################################
diff  --git a/lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.cpp b/lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.cpp
index 82f21fef22156c..931d7d1f1b7bd6 100644
--- a/lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.cpp
+++ b/lldb/source/Plugins/DynamicLoader/Darwin-Kernel/DynamicLoaderDarwinKernel.cpp
@@ -909,10 +909,8 @@ bool DynamicLoaderDarwinKernel::KextImageInfo::LoadImageUsingMemoryModule(
                 ondisk_section_list->GetSectionAtIndex(sect_idx));
             if (ondisk_section_sp) {
               // Don't ever load __LINKEDIT as it may or may not be actually
-              // mapped into memory and there is no current way to tell.
-              // I filed rdar://problem/12851706 to track being able to tell
-              // if the __LINKEDIT is actually mapped, but until then, we need
-              // to not load the __LINKEDIT
+              // mapped into memory and there is no current way to tell. Until
+              // such an ability exists, do not load the __LINKEDIT.
               if (ignore_linkedit &&
                   ondisk_section_sp->GetName() == g_section_name_LINKEDIT)
                 continue;


        


More information about the lldb-commits mailing list