Use interned Names in NoExitRuntime (#1837)

Author: kripken

src/passes/NoExitRuntime.cpp

@@ -34,17 +34,17 @@ struct NoExitRuntime : public WalkerPass<PostWalker<NoExitRuntime>> {
 
   Pass* create() override { return new NoExitRuntime; }
 
+  // Remove all possible manifestations of atexit, across asm2wasm and llvm wasm backend.
+  std::array<Name, 4> ATEXIT_NAMES = {{ "___cxa_atexit",
+                                        "__cxa_atexit",
+                                        "_atexit",
+                                        "atexit" }};
+
   void visitCall(Call* curr) {
     auto* import = getModule()->getFunctionOrNull(curr->target);
     if (!import || !import->imported() || import->module != ENV) return;
-    // Remove all possible manifestations of atexit, across asm2wasm and llvm wasm backend.
-    for (auto* name : {
-      "___cxa_atexit",
-      "_atexit",
-      "__cxa_atexit",
-      "atexit",
-    }) {
-      if (strcmp(name, import->base.str) == 0) {
+    for (auto name : ATEXIT_NAMES) {
+      if (name == import->base) {
         replaceCurrent(
           Builder(*getModule()).replaceWithIdenticalType(curr)
         );