[PATCH] D65988: [Clang Tablegen] Fix build when std::string::end() is not assignable

[Orivej Desh via Phabricator via cfe-commits]

orivej created this revision.
orivej added a reviewer: clang.
orivej added a project: clang.
Herald added subscribers: cfe-commits, kadircet, arphaman.

This fixes Clang error: expression is not assignable:

  Result.erase(--Result.end());                                                 
       
               ^ ~~~~~~~~~~~~                                                   
       


Repository:
  rC Clang

https://reviews.llvm.org/D65988

Files:
  utils/TableGen/ClangDiagnosticsEmitter.cpp


Index: utils/TableGen/ClangDiagnosticsEmitter.cpp
===================================================================
--- utils/TableGen/ClangDiagnosticsEmitter.cpp
+++ utils/TableGen/ClangDiagnosticsEmitter.cpp
@@ -950,7 +950,7 @@
         Result += '|';
       }
       if (!P->Options.empty())
-        Result.erase(--Result.end());
+        Result.pop_back();
       Result += '}';
     }
     addInt(mapIndex(P->Index));
@@ -966,7 +966,7 @@
       Result += "|";
     }
     if (!P->Options.empty())
-      Result.erase(--Result.end());
+      Result.pop_back();
     Result += '}';
     addInt(mapIndex(P->Index));
   }



Index: utils/TableGen/ClangDiagnosticsEmitter.cpp
===================================================================
--- utils/TableGen/ClangDiagnosticsEmitter.cpp
+++ utils/TableGen/ClangDiagnosticsEmitter.cpp
@@ -950,7 +950,7 @@
         Result += '|';
       }
       if (!P->Options.empty())
-        Result.erase(--Result.end());
+        Result.pop_back();
       Result += '}';
     }
     addInt(mapIndex(P->Index));
@@ -966,7 +966,7 @@
       Result += "|";
     }
     if (!P->Options.empty())
-      Result.erase(--Result.end());
+      Result.pop_back();
     Result += '}';
     addInt(mapIndex(P->Index));
   }

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits