[llvm-commits] [llvm] r94412 - /llvm/trunk/lib/Target/X86/X86MCAsmInfo.cpp

[Chris Lattner]

Author: lattner
Date: Mon Jan 25 01:26:17 2010
New Revision: 94412

URL: http://llvm.org/viewvc/llvm-project?rev=94412&view=rev
Log:
Remove support for i386 tiger tools for aligned common symbols.
Previously we would just silently miscompile code that used aligned
common's, now at least you'll get a build error.  tiger-ppc already
triggered the build error because it didn't have a version of this
logic.

Modified:
    llvm/trunk/lib/Target/X86/X86MCAsmInfo.cpp

Modified: llvm/trunk/lib/Target/X86/X86MCAsmInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86MCAsmInfo.cpp?rev=94412&r1=94411&r2=94412&view=diff

==============================================================================
--- llvm/trunk/lib/Target/X86/X86MCAsmInfo.cpp (original)
+++ llvm/trunk/lib/Target/X86/X86MCAsmInfo.cpp Mon Jan 25 01:26:17 2010
@@ -55,9 +55,6 @@
   if (!is64Bit)
     Data64bitsDirective = 0;       // we can't emit a 64-bit unit
 
-  // Leopard and above support aligned common symbols.
-  COMMDirectiveTakesAlignment = Triple.getDarwinMajorNumber() >= 9;
-
   CommentString = "##";
   PCSymbol = ".";