[PATCH] D94677: [lld][WebAssembly] Common superclass for input globals/events/tables

[Sam Clegg via Phabricator via llvm-commits]

sbc100 added inline comments.


================
Comment at: lld/wasm/InputDefinition.h:25
+protected:
+  explicit InputDefinition(StringRef name, ObjFile *f)
+      : file(f), live(!config->gcSections), name(name) {}
----------------
Is the explicit keyword useful here?  I'm used to seeing it mostly on single-arg constructors.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94677