[llvm] 4e844a1 - [WebAssembly] Replace Bugzilla links with Github issues

Heejin Ahn via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 17 20:13:38 PDT 2023


Author: Heejin Ahn
Date: 2023-03-17T20:13:00-07:00
New Revision: 4e844a1498820a98eadca54662165a0ac5287439

URL: https://github.com/llvm/llvm-project/commit/4e844a1498820a98eadca54662165a0ac5287439
DIFF: https://github.com/llvm/llvm-project/commit/4e844a1498820a98eadca54662165a0ac5287439.diff

LOG: [WebAssembly] Replace Bugzilla links with Github issues

Reviewed By: dschuff, asb

Differential Revision: https://reviews.llvm.org/D145966

Added: 
    

Modified: 
    lld/test/wasm/lto/pic-empty.s
    lld/wasm/InputFiles.cpp
    lld/wasm/Writer.cpp
    llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp
    llvm/lib/Target/WebAssembly/WebAssemblyNullifyDebugValueLists.cpp
    llvm/test/CodeGen/WebAssembly/lower-em-exceptions.ll
    llvm/test/MC/WebAssembly/section-symbol.s

Removed: 
    


################################################################################
diff  --git a/lld/test/wasm/lto/pic-empty.s b/lld/test/wasm/lto/pic-empty.s
index 51938e1ad429d..4fe4dffda1dcb 100644
--- a/lld/test/wasm/lto/pic-empty.s
+++ b/lld/test/wasm/lto/pic-empty.s
@@ -3,7 +3,7 @@
 ; 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

diff  --git a/lld/wasm/InputFiles.cpp b/lld/wasm/InputFiles.cpp
index e8a3701232fb1..75760293bbaae 100644
--- a/lld/wasm/InputFiles.cpp
+++ b/lld/wasm/InputFiles.cpp
@@ -441,7 +441,7 @@ void ObjFile::parse(bool ignoreComdats) {
   // 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);
@@ -791,7 +791,8 @@ void BitcodeFile::parse() {
   }
   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));
 

diff  --git a/lld/wasm/Writer.cpp b/lld/wasm/Writer.cpp
index 44f90f34faffd..030ef7468791a 100644
--- a/lld/wasm/Writer.cpp
+++ b/lld/wasm/Writer.cpp
@@ -567,7 +567,7 @@ void Writer::populateTargetFeatures() {
   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");
   }
 

diff  --git a/llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp b/llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp
index 82b016df2f648..0e03890088ef0 100644
--- a/llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp
+++ b/llvm/lib/Target/WebAssembly/WebAssemblyLowerEmscriptenEHSjLj.cpp
@@ -1217,7 +1217,7 @@ bool WebAssemblyLowerEmscriptenEHSjLj::runEHOnFunction(Function &F) {
     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);
     }

diff  --git a/llvm/lib/Target/WebAssembly/WebAssemblyNullifyDebugValueLists.cpp b/llvm/lib/Target/WebAssembly/WebAssemblyNullifyDebugValueLists.cpp
index 5d8c58dcc3347..9b94ee15253d4 100644
--- a/llvm/lib/Target/WebAssembly/WebAssemblyNullifyDebugValueLists.cpp
+++ b/llvm/lib/Target/WebAssembly/WebAssemblyNullifyDebugValueLists.cpp
@@ -9,7 +9,7 @@
 /// \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
 ///
 //===----------------------------------------------------------------------===//

diff  --git a/llvm/test/CodeGen/WebAssembly/lower-em-exceptions.ll b/llvm/test/CodeGen/WebAssembly/lower-em-exceptions.ll
index 1d75bafb102ab..d17a5b419e351 100644
--- a/llvm/test/CodeGen/WebAssembly/lower-em-exceptions.ll
+++ b/llvm/test/CodeGen/WebAssembly/lower-em-exceptions.ll
@@ -69,7 +69,7 @@ catch:                                            ; preds = %catch.dispatch
 ; 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:

diff  --git a/llvm/test/MC/WebAssembly/section-symbol.s b/llvm/test/MC/WebAssembly/section-symbol.s
index b55221c0947f3..adf6111a02b0d 100644
--- a/llvm/test/MC/WebAssembly/section-symbol.s
+++ b/llvm/test/MC/WebAssembly/section-symbol.s
@@ -3,7 +3,7 @@
 # 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


        


More information about the llvm-commits mailing list