[llvm-commits] [dragonegg] r137951 - /dragonegg/trunk/src/Types.cpp

[Duncan Sands]

Author: baldrick
Date: Thu Aug 18 12:23:34 2011
New Revision: 137951

URL: http://llvm.org/viewvc/llvm-project?rev=137951&view=rev
Log:
Disable an extra check I added this morning - it fails when
building GCC.

Modified:
    dragonegg/trunk/src/Types.cpp

Modified: dragonegg/trunk/src/Types.cpp
URL: http://llvm.org/viewvc/llvm-project/dragonegg/trunk/src/Types.cpp?rev=137951&r1=137950&r2=137951&view=diff
==============================================================================
--- dragonegg/trunk/src/Types.cpp (original)
+++ dragonegg/trunk/src/Types.cpp Thu Aug 18 12:23:34 2011
@@ -350,15 +350,17 @@
     }
   }
   // Check that the LLVM type has the same alignment or less than the GCC type.
-  if (Ty->isSized()) {
-    unsigned GCCAlign = TYPE_ALIGN(Tr);
-    unsigned LLVMAlign = getTargetData().getABITypeAlignment(Ty) * 8;
-    if (LLVMAlign > GCCAlign) {
-      errs() << "GCC align: " << GCCAlign << "; LLVM align: " << LLVMAlign
-        << "\n";
-      Mismatch = true;
-    }
-  }
+// FIXME: Reduce LLVM array alignment when the GCC array has a small alignment
+// (due to an alignment clause?), then turn this back on.
+//  if (Ty->isSized()) {
+//    unsigned GCCAlign = TYPE_ALIGN(Tr);
+//    unsigned LLVMAlign = getTargetData().getABITypeAlignment(Ty) * 8;
+//    if (LLVMAlign > GCCAlign) {
+//      errs() << "GCC align: " << GCCAlign << "; LLVM align: " << LLVMAlign
+//        << "\n";
+//      Mismatch = true;
+//    }
+//  }
   if (Mismatch) {
     errs() << "GCC: ";
     debug_tree(Tr);