[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