[PATCH] D43416: [WebAssembly] Simplify FunctionSymbol get/set/hasFunctionType. NFC.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 16 15:42:33 PST 2018


ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: wasm/Symbols.h:124-125
 public:
   DefinedFunction(StringRef Name, uint32_t Flags, InputFile *F = nullptr,
-                  InputChunk *C = nullptr)
-      : FunctionSymbol(Name, DefinedFunctionKind, Flags, F, C) {}
+                  InputFunction *Function = nullptr)
+      : FunctionSymbol(Name, DefinedFunctionKind, Flags, F, Function) {}
----------------
If you do not `= nullptr`, please remove it as well.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D43416





More information about the llvm-commits mailing list