From 0ec9ddf416de2dc9fc68d574513c52079c7ab238 Mon Sep 17 00:00:00 2001 From: Alon Zakai Date: Wed, 31 Jul 2019 20:33:26 -0700 Subject: Proper Asyncify list name handling (#2275) The lists are comma separated, but the names can have internal commas since they are human-readable. This adds awareness of bracketing things, so void foo(int, double) is parsed as a single function name, properly. Helps emscripten-core/emscripten#9128 --- src/passes/Asyncify.cpp | 3 +++ 1 file changed, 3 insertions(+) (limited to 'src/passes/Asyncify.cpp') diff --git a/src/passes/Asyncify.cpp b/src/passes/Asyncify.cpp index 361976fc3..379452d42 100644 --- a/src/passes/Asyncify.cpp +++ b/src/passes/Asyncify.cpp @@ -1043,6 +1043,9 @@ struct Asyncify : public Pass { String::Split whitelist( runner->options.getArgumentOrDefault("asyncify-whitelist", ""), ","); + blacklist = handleBracketingOperators(blacklist); + whitelist = handleBracketingOperators(whitelist); + // The lists contain human-readable strings. Turn them into the internal // escaped names for later comparisons auto processList = [module](String::Split& list, const std::string& which) { -- cgit v1.2.3