[PATCH] D126976: [lld][WebAssembly][NFC] Move getOutputSegmentName to InputChunk methods

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 3 10:41:51 PDT 2022


sbc100 added inline comments.


================
Comment at: lld/wasm/InputChunks.cpp:531
+    return ".tdata";
+  StringRef name = getName();
+  if (!config->mergeDataSegments)
----------------
Actually, looking at this code and looking at how ELF does this.. I think you can just use `getName` in the other patch and maybe this patch is not actually needed?

None of these combined segments return true for `isValidCIdentifier` so I think you can just to what ELF in its MakeLive.cpp and use the intput section name.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126976



More information about the llvm-commits mailing list