[llvm] [llvm][Support][NFC] Disentangle SpecialCaseList parsing and matching (PR #116800)

Boaz Brickner via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 20 04:17:26 PST 2024


================
@@ -113,22 +120,49 @@ bool SpecialCaseList::createInternal(const std::vector<std::string> &Paths,
       Error = (Twine("can't open file '") + Path + "': " + EC.message()).str();
       return false;
     }
-    std::string ParseError;
-    if (!parse(FileOrErr.get().get(), ParseError)) {
-      Error = (Twine("error parsing file '") + Path + "': " + ParseError).str();
+    if (!createInternal(FileOrErr->get(), Error)) {
+      Error = llvm::formatv("error parsing file '{0}': {1}", Path, Error);
----------------
bricknerb wrote:

This change seems somewhat orthogonal to the disentanglement change.
How about moving it to a separate PR?

https://github.com/llvm/llvm-project/pull/116800


More information about the llvm-commits mailing list