[PATCH] D112667: [lld][WebAssembly] Initialize bss segments using memory.fill

Thomas Lively via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 28 09:48:30 PDT 2021


tlively added inline comments.


================
Comment at: lld/wasm/Writer.cpp:539
+
+void Writer::checkTargetFeatures() {
+  if (config->relocatable || !config->checkFeatures)
----------------
There are still a bunch of other feature checks in the previous function. Would it make sense to break them out as well?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112667



More information about the llvm-commits mailing list