Rename function to avoid function overloading.

We had two functions with the same name to read completely different
things. That was confusing.

llvm-svn: 286991
diff --git a/lld/ELF/LinkerScript.cpp b/lld/ELF/LinkerScript.cpp
index a202dc5..1f65c37 100644
--- a/lld/ELF/LinkerScript.cpp
+++ b/lld/ELF/LinkerScript.cpp
@@ -986,7 +986,7 @@
   Expr readParenExpr();
 
   // For parsing version script.
-  void readExtern(std::vector<SymbolVersion> *Globals);
+  void readVersionExtern(std::vector<SymbolVersion> *Globals);
   void readVersionDeclaration(StringRef VerStr);
   void readGlobal(StringRef VerStr);
   void readLocal(StringRef VerStr);
@@ -1824,7 +1824,7 @@
   }
 }
 
-void ScriptParser::readExtern(std::vector<SymbolVersion> *Globals) {
+void ScriptParser::readVersionExtern(std::vector<SymbolVersion> *Globals) {
   expect("\"C++\"");
   expect("{");
 
@@ -1849,7 +1849,7 @@
 
   for (;;) {
     if (consume("extern"))
-      readExtern(Globals);
+      readVersionExtern(Globals);
 
     StringRef Cur = peek();
     if (Cur == "}" || Cur == "local:" || Error)