[llvm] Fix comment in wasm unreachable test (PR #70340)
via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 26 07:53:24 PDT 2023
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-backend-webassembly
Author: Matt Harding (majaha)
<details>
<summary>Changes</summary>
Some textual editing errors got through this pull request that was merged a few weeks ago: https://github.com/llvm/llvm-project/pull/65876
This patch clears up the unintentional duplicated line, and white-space at the end of the lines.
---
Full diff: https://github.com/llvm/llvm-project/pull/70340.diff
1 Files Affected:
- (modified) llvm/test/CodeGen/WebAssembly/unreachable.ll (+4-5)
``````````diff
diff --git a/llvm/test/CodeGen/WebAssembly/unreachable.ll b/llvm/test/CodeGen/WebAssembly/unreachable.ll
index 72f865842bdcaa5..5368c2ba5b8dc1c 100644
--- a/llvm/test/CodeGen/WebAssembly/unreachable.ll
+++ b/llvm/test/CodeGen/WebAssembly/unreachable.ll
@@ -80,11 +80,10 @@ define i32 @missing_ret_unreachable() {
unreachable
}
-; This is similar to the above test, but ensures wasm unreachable is emitted
-; This is similar to the above test, but the callee has a 'noreturn' attribute.
-; There is an optimization that removes an 'unreachable' after a noreturn call,
-; but Wasm backend doesn't use it and ignore `--no-trap-after-noreturn`, if
-; given, to generate valid code.
+; This is similar to the above test, but the callee has a 'noreturn' attribute.
+; There is an optimization that removes an 'unreachable' after a noreturn call,
+; but Wasm backend doesn't use it and ignore `--no-trap-after-noreturn`, if
+; given, to generate valid code.
define i32 @missing_ret_noreturn_unreachable() {
; CHECK-LABEL: missing_ret_noreturn_unreachable:
; CHECK: .functype missing_ret_noreturn_unreachable () -> (i32)
``````````
</details>
https://github.com/llvm/llvm-project/pull/70340
More information about the llvm-commits
mailing list