[PATCH] D71793: [WebAssembly] Support wasm exports with zero-length names.

[Sam Clegg via Phabricator via llvm-commits]

sbc100 accepted this revision.
sbc100 added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lld/wasm/InputChunks.h:134
+  llvm::Optional<StringRef> getExportName() const {
+    return function ? function->ExportName : llvm::Optional<StringRef>();
   }
----------------
Is there a shorter way to write that?  Does just {} work?


================
Comment at: llvm/include/llvm/BinaryFormat/Wasm.h:182
   StringRef ImportModule; // For undefined symbols the module of the import
   StringRef ImportName;   // For undefined symbols the name of the import
+  Optional<StringRef> ExportName;   // For symbols to be exported from the final module
----------------
Would it make sense to do the same for there two too?  Or is that a followup?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71793/new/

https://reviews.llvm.org/D71793