[ELF] - replaced std::function with raw pointers in LinkerScript.cpp

[grimar]

Summary

Little change as was proposed by David Blaikie.

Rui, Rafael, what do you think ?

[• rafael]

LGTM

[dblaikie]

ELF/LinkerScript.cpp

120

Should this be static/const, then? (doesn't seem it ever gets modified, etc)

[grimar]

Addressed David's and Rui's review comments.

[ruiu]

LGTM assuming you address David Blaikie's comment.

ELF/LinkerScript.cpp

static bool matchStr(StringRef S, StringRef T) {
   }
 }
 
 template <class ELFT> bool SectionRule::match(InputSectionBase<ELFT> *S) {
   return matchStr(SectionPattern, S->getSectionName());
 }
 
 class elf2::ScriptParser {
+  typedef void (ScriptParser::*CmdHandler)();
+
 public:
   ScriptParser(BumpPtrAllocator *A, StringRef S, bool B)
-      : Saver(*A), Tokens(tokenize(S)), IsUnderSysroot(B) {
-    Cmd["ENTRY"] = std::mem_fn(&ScriptParser::readEntry);
-    Cmd["EXTERN"] = std::mem_fn(&ScriptParser::readExtern);
-    Cmd["GROUP"] = std::mem_fn(&ScriptParser::readGroup);
-    Cmd["INCLUDE"] = std::mem_fn(&ScriptParser::readInclude);
-    Cmd["INPUT"] = std::mem_fn(&ScriptParser::readGroup);
-    Cmd["OUTPUT"] = std::mem_fn(&ScriptParser::readOutput);
-    Cmd["OUTPUT_ARCH"] = std::mem_fn(&ScriptParser::readOutputArch);
-    Cmd["OUTPUT_FORMAT"] = std::mem_fn(&ScriptParser::readOutputFormat);
-    Cmd["SEARCH_DIR"] = std::mem_fn(&ScriptParser::readSearchDir);
-    Cmd["SECTIONS"] = std::mem_fn(&ScriptParser::readSections);
-    Cmd[";"] = [](ScriptParser &) {};
-  }
+      : Saver(*A), Tokens(tokenize(S)), IsUnderSysroot(B) {}
 
   void run();
 
 private:
   void setError(const Twine &Msg);
   static std::vector<StringRef> tokenize(StringRef S);
   static StringRef skipSpace(StringRef S);
   bool atEOF();
private:
   void readOutputFormat();
   void readSearchDir();
   void readSections();
 
   void readOutputSectionDescription();
 
   StringSaver Saver;
   std::vector<StringRef> Tokens;
-  llvm::StringMap<std::function<void(ScriptParser &)>> Cmd;
+  llvm::StringMap<CmdHandler> Cmd{
+      {"ENTRY", &ScriptParser::readEntry},
+      {"EXTERN", &ScriptParser::readExtern},
+      {"GROUP", &ScriptParser::readGroup},
+      {"INCLUDE", &ScriptParser::readInclude},
+      {"INPUT", &ScriptParser::readGroup},
+      {"OUTPUT", &ScriptParser::readOutput},
+      {"OUTPUT_ARCH", &ScriptParser::readOutputArch},
+      {"OUTPUT_FORMAT", &ScriptParser::readOutputFormat},
+      {"SEARCH_DIR", &ScriptParser::readSearchDir},
+      {"SECTIONS", &ScriptParser::readSections},
+      {";", nullptr}};
   size_t Pos = 0;
   bool IsUnderSysroot;
   bool Error = false;
 };
 
 void ScriptParser::run() {
   while (!atEOF()) {
     StringRef Tok = next();
     auto It = Cmd.find(Tok);
     if (It != Cmd.end()) {
-      std::function<void(ScriptParser &)> &Handler = It->second;
-      Handler(*this);
+      CmdHandler Handler = It->second;
+      if (Handler)
+        (this->*Handler)();
     } else {
       setError("unknown directive: " + Tok);
     }
   }
 }
 
 // We don't want to record cascading errors. Keep only the first one.
 void ScriptParser::setError(const Twine &Msg) {