[PATCH] D23829: [ELF] - Use std::regex instead of hand written logic in elf::globMatch()

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 31 07:27:18 PDT 2016


ruiu added inline comments.

================
Comment at: ELF/LinkerScript.cpp:907-912
@@ -903,8 +906,8 @@
 
-std::vector<StringRef> ScriptParser::readInputFilePatterns() {
-  std::vector<StringRef> V;
+std::vector<Regex> ScriptParser::readInputFilePatterns() {
+  std::vector<Regex> V;
   while (!Error && !skip(")"))
-    V.push_back(next());
+    V.push_back(compileGlobPattern(next()));
   return V;
 }
 
----------------
Why don't you add a constructor with no arguments to llvm::Regex?


https://reviews.llvm.org/D23829





More information about the llvm-commits mailing list