[llvm-commits] CVS: llvm/lib/Target/TargetData.cpp

Reid Spencer reid at x10sys.com
Mon Feb 19 15:30:27 PST 2007



Changes in directory llvm/lib/Target:

TargetData.cpp updated: 1.102 -> 1.103
---
Log message:

Simplify some code by moving variable declarations into the only block that
uses them.


---
Diffs of the changes:  (+6 -12)

 TargetData.cpp |   18 ++++++------------
 1 files changed, 6 insertions(+), 12 deletions(-)


Index: llvm/lib/Target/TargetData.cpp
diff -u llvm/lib/Target/TargetData.cpp:1.102 llvm/lib/Target/TargetData.cpp:1.103
--- llvm/lib/Target/TargetData.cpp:1.102	Mon Feb 19 16:35:00 2007
+++ llvm/lib/Target/TargetData.cpp	Mon Feb 19 17:30:10 2007
@@ -184,14 +184,8 @@
   
   while (!temp.empty()) {
     std::string token = getToken(temp, "-");
-    
     std::string arg0 = getToken(token, ":");
     const char *p = arg0.c_str();
-    AlignTypeEnum align_type;
-    uint32_t size;
-    unsigned char abi_align;
-    unsigned char pref_align;
-
     switch(*p) {
     case 'E':
       LittleEndian = false;
@@ -210,12 +204,12 @@
     case 'v':
     case 'f':
     case 'a': {
-      align_type = (*p == 'i' ? INTEGER_ALIGN :
-                    (*p == 'f' ? FLOAT_ALIGN :
-                     (*p == 'v' ? VECTOR_ALIGN : AGGREGATE_ALIGN)));
-      size = (uint32_t) atoi(++p);
-      abi_align = atoi(getToken(token, ":").c_str()) / 8;
-      pref_align = atoi(getToken(token, ":").c_str()) / 8;
+      AlignTypeEnum align_type = 
+        (*p == 'i' ? INTEGER_ALIGN : (*p == 'f' ? FLOAT_ALIGN :
+           (*p == 'v' ? VECTOR_ALIGN : AGGREGATE_ALIGN)));
+      uint32_t size = (uint32_t) atoi(++p);
+      unsigned char abi_align = atoi(getToken(token, ":").c_str()) / 8;
+      unsigned char pref_align = atoi(getToken(token, ":").c_str()) / 8;
       if (pref_align == 0)
         pref_align = abi_align;
       setAlignment(align_type, abi_align, pref_align, size);






More information about the llvm-commits mailing list