[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 §ion = 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