commit 58a29128005f6e54c7d3aa39797d86ada8d40006
author Daniel Dunbar <daniel@zuster.org> Mon Mar 09 22:18:41 2009 +0000
committer Daniel Dunbar <daniel@zuster.org> Mon Mar 09 22:18:41 2009 +0000
tree 9e0f5d4f730b25c6c580fc9acbd1743069fb733c
parent 151048812a8b4486f842108f1e3358b0dc6cc552

NeXT: Set alignment on a number of ObjC metadata variables (matching llvm-gcc).


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@66481 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/CGObjCMac.cpp

diff --git a/lib/CodeGen/CGObjCMac.cpp b/lib/CodeGen/CGObjCMac.cpp
index 454f663..d33005e 100644
--- a/lib/CodeGen/CGObjCMac.cpp
+++ b/lib/CodeGen/CGObjCMac.cpp
@@ -1086,6 +1086,7 @@
                                std::string("\01L_OBJC_PROTOCOL_")+ProtocolName,
                                &CGM.getModule());
     Entry->setSection("__OBJC,__protocol,regular,no_dead_strip");
+    Entry->setAlignment(4);
     UsedGlobals.push_back(Entry);
     // FIXME: Is this necessary? Why only for protocol?
     Entry->setAlignment(4);
@@ -1108,6 +1109,7 @@
                                "\01L_OBJC_PROTOCOL_" + PD->getNameAsString(),
                                &CGM.getModule());
     Entry->setSection("__OBJC,__protocol,regular,no_dead_strip");
+    Entry->setAlignment(4);
     UsedGlobals.push_back(Entry);
     // FIXME: Is this necessary? Why only for protocol?
     Entry->setAlignment(4);
@@ -1195,7 +1197,7 @@
   llvm::Constant *Init = llvm::ConstantStruct::get(Values);
   llvm::GlobalVariable *GV = 
     CreateMetadataVar(Name, Init, "__OBJC,__cat_cls_meth,regular,no_dead_strip",
-                      0, false);
+                      4, false);
   return llvm::ConstantExpr::getBitCast(GV, ObjCTypes.ProtocolListPtrTy);
 }
 
@@ -1354,7 +1356,7 @@
   llvm::GlobalVariable *GV = 
     CreateMetadataVar(std::string("\01L_OBJC_CATEGORY_")+ExtName, Init,
                       "__OBJC,__category,regular,no_dead_strip",
-                      0, true);
+                      4, true);
   DefinedCategories.push_back(GV);
 }
 
@@ -1489,8 +1491,7 @@
   llvm::GlobalVariable *GV = 
     CreateMetadataVar(std::string("\01L_OBJC_CLASS_")+ClassName, Init,
                       "__OBJC,__class,regular,no_dead_strip",
-                      32, // FIXME: Why?
-                      true);
+                      4, true);
   DefinedClasses.push_back(GV);
 }
 
@@ -1559,9 +1560,8 @@
                                   &CGM.getModule());
   }
   GV->setSection("__OBJC,__meta_class,regular,no_dead_strip");
+  GV->setAlignment(4);
   UsedGlobals.push_back(GV);
-  // FIXME: Why?
-  GV->setAlignment(32);
 
   return GV;
 }
@@ -1715,8 +1715,8 @@
   llvm::GlobalVariable *GV;
   if (ForClass)
     GV = CreateMetadataVar("\01L_OBJC_CLASS_VARIABLES_" + ID->getNameAsString(),
-                           Init, "__OBJC,__cls_vars,regular,no_dead_strip", 
-                           32, true);
+                           Init, "__OBJC,__class_vars,regular,no_dead_strip", 
+                           4, true);
   else
     GV = CreateMetadataVar("\01L_OBJC_INSTANCE_VARIABLES_"
                            + ID->getNameAsString(),
@@ -2349,7 +2349,7 @@
   CreateMetadataVar("\01L_OBJC_MODULES", 
                     llvm::ConstantStruct::get(ObjCTypes.ModuleTy, Values),
                     "__OBJC,__module_info,regular,no_dead_strip",
-                    0, true);
+                    4, true);
 }
 
 llvm::Constant *CGObjCMac::EmitModuleSymbols() {
@@ -3493,8 +3493,7 @@
                                Init,
                                "\01L_OBJC_LABEL_CLASS_$",
                                &CGM.getModule());
-    GV->setAlignment(
-      CGM.getTargetData().getPrefTypeAlignment(ObjCTypes.Int8PtrTy));
+    GV->setAlignment(8);
     GV->setSection("__DATA, __objc_classlist, regular, no_dead_strip");
     UsedGlobals.push_back(GV);
   }
@@ -3521,6 +3520,7 @@
                                Init,
                                "\01L_OBJC_LABEL_CATEGORY_$",
                                &CGM.getModule());
+    GV->setAlignment(8);
     GV->setSection("__DATA, __objc_catlist, regular, no_dead_strip");
     UsedGlobals.push_back(GV);
   }