[llvm-commits] [llvm] r143472 - /llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldMachO.cpp

Jim Grosbach grosbach at apple.com
Tue Nov 1 11:10:23 PDT 2011


Author: grosbach
Date: Tue Nov  1 13:10:23 2011
New Revision: 143472

URL: http://llvm.org/viewvc/llvm-project?rev=143472&view=rev
Log:
Ignore MachO symbol flags in the upper nibble of n_desc.

They don't impact the MCJIT rtdyld, so just mask them off for now.

Modified:
    llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldMachO.cpp

Modified: llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldMachO.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldMachO.cpp?rev=143472&r1=143471&r2=143472&view=diff
==============================================================================
--- llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldMachO.cpp (original)
+++ llvm/trunk/lib/ExecutionEngine/RuntimeDyld/RuntimeDyldMachO.cpp Tue Nov  1 13:10:23 2011
@@ -176,7 +176,8 @@
         continue;
       // Flags == 0x8 marks a thumb function for ARM, which is fine as it
       // doesn't require any special handling here.
-      if (STE->Flags != 0x0 && STE->Flags != 0x8)
+      // Flags in the upper nibble we don't care about.
+      if ((STE->Flags & 0xf) != 0x0 && STE->Flags != 0x8)
         continue;
 
       // Remember the symbol.
@@ -313,7 +314,8 @@
       // FIXME: Check the symbol type and flags.
       if (STE->Type != 0xF)  // external, defined in this section.
         continue;
-      if (STE->Flags != 0x0)
+      // Flags in the upper nibble we don't care about.
+      if ((STE->Flags & 0xf) != 0x0)
         continue;
 
       // Remember the symbol.





More information about the llvm-commits mailing list