[llvm-commits] [llvm] r79683 - /llvm/trunk/tools/llvm-mc/AsmParser.cpp

[Daniel Dunbar]

Author: ddunbar
Date: Fri Aug 21 18:01:53 2009
New Revision: 79683

URL: http://llvm.org/viewvc/llvm-project?rev=79683&view=rev
Log:
llvm-mc: In a .fill directive, still honor .align even if invalid maximum bytes
count is given (this matches 'as').

Modified:
    llvm/trunk/tools/llvm-mc/AsmParser.cpp

Modified: llvm/trunk/tools/llvm-mc/AsmParser.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-mc/AsmParser.cpp?rev=79683&r1=79682&r2=79683&view=diff

==============================================================================
--- llvm/trunk/tools/llvm-mc/AsmParser.cpp (original)
+++ llvm/trunk/tools/llvm-mc/AsmParser.cpp Fri Aug 21 18:01:53 2009
@@ -1072,12 +1072,13 @@
     Alignment = 1LL << Alignment;
   }
 
-  // Diagnose non-sensical max bytes to fill.
+  // Diagnose non-sensical max bytes to fill, which are treated as missing (this
+  // matches 'as').
   if (MaxBytesLoc.isValid()) {
     if (MaxBytesToFill < 1) {
       Warning(MaxBytesLoc, "alignment directive can never be satisfied in this "
-              "many bytes, ignoring");
-      return false;
+              "many bytes, ignoring maximum bytes expression");
+      MaxBytesToFill = 0;
     }
 
     if (MaxBytesToFill >= Alignment) {