[WebAssembly] Replace Bugzilla links with Github issues

[asb]

LGTM.

[aheejin]

Replace more bugzilla links in lld

[aheejin]

Replace more bugzilla links in llvm/test

[aheejin]

lld/wasm/InputFiles.cpp

444

@sbc100 This issue seems to be closed. Do we still need this comment?

[sbc100]

lld/wasm/InputFiles.cpp

444

I think this was added for context, rather than as TODO/FIXME. So I think we can keep it.

lld/test/wasm/lto/pic-empty.s

 ; Test that LTO objects with no function can still be linked as shared
 ; libraries.
 ; We had a bug where the mutable-globals feature was not being added
 ; so the linker-generated import of `__stack_pointer` (which is currently
 ; mandatory for ; shared libraries) was generating a linker error.
-; See https://bugs.llvm.org/show_bug.cgi?id=52339
+; See https://github.com/llvm/llvm-project/issues/51681.
 
 ; RUN: llvm-as %s -o %t.o
 ; RUN: wasm-ld --lto-O2 --experimental-pic -shared --no-gc-sections --export=tls_int %t.o -o %t.so
 ; RUN: obj2yaml %t.so  | FileCheck %s
 
 target datalayout = "e-m:e-p:32:32-p10:8:8-p20:8:8-i64:64-f128:64-n32:64-S128-ni:1:10:20"
 target triple = "wasm32-unknown-emscripten"
 
 @tls_int = dso_local thread_local global i32 99
 
 ; CHECK:  - Type:            CUSTOM
 ; CHECK:    Name:            target_features
 ; CHECK:    Features:
 ; CHECK:      - Prefix:          USED
 ; CHECK:        Name:            mutable-globals

lld/wasm/InputFiles.cpp

void ObjFile::parse(bool ignoreComdats) {
 
   uint32_t sectionIndex = 0;
 
   // Bool for each symbol, true if called directly.  This allows us to implement
   // a weaker form of signature checking where undefined functions that are not
   // called directly (i.e. only address taken) don't have to match the defined
   // function's signature.  We cannot do this for directly called functions
   // because those signatures are checked at validation times.
-  // See https://bugs.llvm.org/show_bug.cgi?id=40412
+  // See https://github.com/llvm/llvm-project/issues/39758
   std::vector<bool> isCalledDirectly(wasmObj->getNumberOfSymbols(), false);
   for (const SectionRef &sec : wasmObj->sections()) {
     const WasmSection &section = wasmObj->getWasmSection(sec);
     // Wasm objects can have at most one code and one data section.
     if (section.Type == WASM_SEC_CODE) {
       assert(!codeSection);
       codeSection = &section;
     } else if (section.Type == WASM_SEC_DATA) {
void BitcodeFile::parse() {
 
   Triple t(obj->getTargetTriple());
   if (!t.isWasm()) {
     error(toString(this) + ": machine type must be wasm32 or wasm64");
     return;
   }
   checkArch(t.getArch());
   std::vector<bool> keptComdats;
-  // TODO Support nodeduplicate https://bugs.llvm.org/show_bug.cgi?id=50531
+  // TODO Support nodeduplicate
+  // https://github.com/llvm/llvm-project/issues/49875
   for (std::pair<StringRef, Comdat::SelectionKind> s : obj->getComdatTable())
     keptComdats.push_back(symtab->addComdat(s.first));
 
   for (const lto::InputFile::Symbol &objSym : obj->symbols())
     symbols.push_back(createBitcodeSymbol(keptComdats, objSym, *this));
 }
 
 } // namespace wasm
 } // namespace lld

lld/wasm/Writer.cpp

void Writer::populateTargetFeatures() {
   StringMap<std::string> required;
   StringMap<std::string> disallowed;
   SmallSet<std::string, 8> &allowed = out.targetFeaturesSec->features;
   bool tlsUsed = false;
 
   if (config->isPic) {
     // This should not be necessary because all PIC objects should
     // contain the mutable-globals feature.
-    // TODO(https://bugs.llvm.org/show_bug.cgi?id=52339)
+    // TODO (https://github.com/llvm/llvm-project/issues/51681)
     allowed.insert("mutable-globals");
   }
 
   if (config->extraFeatures.has_value()) {
     auto &extraFeatures = *config->extraFeatures;
     allowed.insert(extraFeatures.begin(), extraFeatures.end());
   }
 

llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp

bool WebAssemblyLowerEmscriptenEHSjLj::runEHOnFunction(Function &F) {
   // Handle all the landingpad for this function together, as multiple invokes
   // may share a single lp
   for (LandingPadInst *LPI : LandingPads) {
     IRB.SetInsertPoint(LPI);
     SmallVector<Value *, 16> FMCArgs;
     for (unsigned I = 0, E = LPI->getNumClauses(); I < E; ++I) {
       Constant *Clause = LPI->getClause(I);
       // TODO Handle filters (= exception specifications).
-      // https://bugs.llvm.org/show_bug.cgi?id=50396
+      // https://github.com/llvm/llvm-project/issues/49740
       if (LPI->isCatch(I))
         FMCArgs.push_back(Clause);
     }
 
     // Create a call to __cxa_find_matching_catch_N function
     Function *FMCF = getFindMatchingCatch(M, FMCArgs.size());
     CallInst *FMCI = IRB.CreateCall(FMCF, FMCArgs, "fmc");
     Value *Poison = PoisonValue::get(LPI->getType());

llvm/lib/Target/WebAssembly/WebAssemblyNullifyDebugValueLists.cpp

 //=== WebAssemblyNullifyDebugValueLists.cpp - Nullify DBG_VALUE_LISTs   ---===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 ///
 /// \file
 /// Nullify DBG_VALUE_LISTs instructions as a temporary measure before we
 /// implement DBG_VALUE_LIST handling in WebAssemblyDebugValueManager.
-/// See https://bugs.llvm.org/show_bug.cgi?id=50361.
+/// See https://github.com/llvm/llvm-project/issues/49705.
 /// TODO Correctly handle DBG_VALUE_LISTs
 ///
 //===----------------------------------------------------------------------===//
 
 #include "WebAssembly.h"
 #include "WebAssemblySubtarget.h"
 #include "llvm/CodeGen/MachineFunctionPass.h"
 using namespace llvm;

llvm/test/CodeGen/WebAssembly/lower-em-exceptions.ll

catch:                                            ; preds = %catch.dispatch
   %6 = call ptr @__cxa_begin_catch(ptr %1)
   call void @__cxa_end_catch()
   br label %try.cont
 }
 
 ; Test invoke instruction with filters (functions with throw(...) declaration)
 ; Currently we don't support exception specifications correctly in JS glue code,
 ; so we ignore all filters here.
-; See https://bugs.llvm.org/show_bug.cgi?id=50396.
+; See https://github.com/llvm/llvm-project/issues/49740.
 define void @filter() personality ptr @__gxx_personality_v0 {
 ; CHECK-LABEL: @filter(
 entry:
   invoke void @foo(i32 3)
           to label %invoke.cont unwind label %lpad
 ; CHECK: entry:
 ; CHECK-NEXT: store [[PTR]] 0, ptr @__THREW__
 ; CHECK-NEXT: call cc{{.*}} void @__invoke_void_i32(ptr @foo, i32 3)

llvm/test/MC/WebAssembly/section-symbol.s

 # RUN: llvm-mc -triple=wasm32 < %s | FileCheck %s
 
 # check that we can refer to section symbols of other sections.
 # getWasmSection currently forces the section symbol to have a suffix.
 
-# TODO: fix the 0-suffix: https://bugs.llvm.org/show_bug.cgi?id=49252
+# TODO: fix the 0-suffix: https://github.com/llvm/llvm-project/issues/48596
 
     .section    .debug_abbrev,"",@
     .int8       1
     .section    .debug_info,"",@
     .int32      .debug_abbrev0
 
 # CHECK:         .section    .debug_abbrev,"",@
 # CHECK-NEXT:    .int8       1
 # CHECK-NEXT:    .section    .debug_info,"",@
 # CHECK-NEXT:    .int32      .debug_abbrev0