[llvm] r194530 - Remove always true flag.

Rafael Espindola rafael.espindola at gmail.com
Tue Nov 12 15:27:08 PST 2013


Author: rafael
Date: Tue Nov 12 17:27:08 2013
New Revision: 194530

URL: http://llvm.org/viewvc/llvm-project?rev=194530&view=rev
Log:
Remove always true flag.

Modified:
    llvm/trunk/include/llvm/MC/MCAsmInfo.h
    llvm/trunk/lib/MC/MCAsmInfo.cpp
    llvm/trunk/lib/Target/Mangler.cpp

Modified: llvm/trunk/include/llvm/MC/MCAsmInfo.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/MC/MCAsmInfo.h?rev=194530&r1=194529&r2=194530&view=diff
==============================================================================
--- llvm/trunk/include/llvm/MC/MCAsmInfo.h (original)
+++ llvm/trunk/include/llvm/MC/MCAsmInfo.h Tue Nov 12 17:27:08 2013
@@ -160,10 +160,6 @@ namespace llvm {
     /// names. Defaults to false.
     bool AllowAtInName;
 
-    /// AllowUTF8 - This is true if the assembler accepts UTF-8 input.
-    // FIXME: Make this a more general encoding setting?
-    bool AllowUTF8;
-
     /// UseDataRegionDirectives - This is true if data region markers should
     /// be printed as ".data_region/.end_data_region" directives. If false,
     /// use "$d/$a" labels instead.
@@ -483,9 +479,6 @@ namespace llvm {
     bool doesAllowAtInName() const {
       return AllowAtInName;
     }
-    bool doesAllowUTF8() const {
-      return AllowUTF8;
-    }
     bool doesSupportDataRegionDirectives() const {
       return UseDataRegionDirectives;
     }

Modified: llvm/trunk/lib/MC/MCAsmInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/MC/MCAsmInfo.cpp?rev=194530&r1=194529&r2=194530&view=diff
==============================================================================
--- llvm/trunk/lib/MC/MCAsmInfo.cpp (original)
+++ llvm/trunk/lib/MC/MCAsmInfo.cpp Tue Nov 12 17:27:08 2013
@@ -54,7 +54,6 @@ MCAsmInfo::MCAsmInfo() {
   AllowNameToStartWithDigit = false;
   AllowPeriodsInName = true;
   AllowAtInName = false;
-  AllowUTF8 = true;
   UseDataRegionDirectives = false;
   ZeroDirective = "\t.zero\t";
   AsciiDirective = "\t.ascii\t";

Modified: llvm/trunk/lib/Target/Mangler.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/Mangler.cpp?rev=194530&r1=194529&r2=194530&view=diff
==============================================================================
--- llvm/trunk/lib/Target/Mangler.cpp (original)
+++ llvm/trunk/lib/Target/Mangler.cpp Tue Nov 12 17:27:08 2013
@@ -23,13 +23,13 @@
 #include "llvm/Support/raw_ostream.h"
 using namespace llvm;
 
-static bool isAcceptableChar(char C, bool AllowPeriod, bool AllowUTF8) {
+static bool isAcceptableChar(char C, bool AllowPeriod) {
   if ((C < 'a' || C > 'z') &&
       (C < 'A' || C > 'Z') &&
       (C < '0' || C > '9') &&
       C != '_' && C != '$' && C != '@' &&
       !(AllowPeriod && C == '.') &&
-      !(AllowUTF8 && (C & 0x80)))
+      !(C & 0x80))
     return false;
   return true;
 }
@@ -58,9 +58,8 @@ static bool NameNeedsEscaping(StringRef
   // If any of the characters in the string is an unacceptable character, force
   // quotes.
   bool AllowPeriod = MAI->doesAllowPeriodsInName();
-  bool AllowUTF8 = MAI->doesAllowUTF8();
   for (unsigned i = 0, e = Str.size(); i != e; ++i)
-    if (!isAcceptableChar(Str[i], AllowPeriod, AllowUTF8))
+    if (!isAcceptableChar(Str[i], AllowPeriod))
       return true;
   return false;
 }
@@ -77,9 +76,8 @@ static void appendMangledName(SmallVecto
   }
 
   bool AllowPeriod = MAI->doesAllowPeriodsInName();
-  bool AllowUTF8 = MAI->doesAllowUTF8();
   for (unsigned i = 0, e = Str.size(); i != e; ++i) {
-    if (!isAcceptableChar(Str[i], AllowPeriod, AllowUTF8))
+    if (!isAcceptableChar(Str[i], AllowPeriod))
       MangleLetter(OutName, Str[i]);
     else
       OutName.push_back(Str[i]);





More information about the llvm-commits mailing list