[lld] wasm-ld: Add allow-multiple-definition flag (PR #97699)

Sam Clegg via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 7 20:52:09 PDT 2024


Martin =?utf-8?q?Žukovec?= <martin.zukovec at xlab.si>,
Martin =?utf-8?q?Žukovec?= <martin.zukovec at xlab.si>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/97699 at github.com>


================
@@ -797,7 +821,7 @@ static void writeWhyExtract() {
 // Equivalent of demote demoteSharedAndLazySymbols() in the ELF linker
 static void demoteLazySymbols() {
   for (Symbol *sym : symtab->symbols()) {
-    if (auto* s = dyn_cast<LazySymbol>(sym)) {
+    if (auto *s = dyn_cast<LazySymbol>(sym)) {
----------------
sbc100 wrote:

Can you revert these hunks that are formatting only (I find it detracts from the substance of a change to mix these in with semantic changes)?

(To format just the parts of the file you touch you can use `git clang-format origin/main`)

https://github.com/llvm/llvm-project/pull/97699


More information about the llvm-commits mailing list