[PATCH] D42271: [WebAssembly] Fix libcall signature lookup

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 18 17:45:55 PST 2018


sbc100 added inline comments.


================
Comment at: lib/Target/WebAssembly/WebAssemblyRuntimeLibcallSignatures.cpp:476
+    if (name && RuntimeLibcallSignatures->Table[RTLIB::code] != unsupported) { \
+      assert(Map.find(StringRefOrEmpty(name)) == Map.end() && "duplicate libcall names in name map"); \
+      Map[StringRefOrEmpty(name)] = RTLIB::code; \
----------------
No need for StringRefOrEmpty anymore since its already inside a null check?


Repository:
  rL LLVM

https://reviews.llvm.org/D42271





More information about the llvm-commits mailing list