]> git.lizzy.rs Git - rust.git/commitdiff
Drop inaccurate comment
authorNikita Popov <npopov@redhat.com>
Tue, 19 Apr 2022 14:12:23 +0000 (16:12 +0200)
committerNikita Popov <npopov@redhat.com>
Wed, 20 Apr 2022 07:25:47 +0000 (09:25 +0200)
The linked issue has an accurate description of the situation,
drop the inaccurate comment.

src/test/codegen/panic-in-drop-abort.rs

index 7b9c41b658ae9d6e68ffd1abee8f22625addd398..7a84484c41996906a75af3073afb4f81be5e17ec 100644 (file)
@@ -6,11 +6,7 @@
 
 // This test uses ignore-msvc, because the expected optimization does not happen on targets using
 // SEH exceptions with the new LLVM pass manager anymore, see
-// https://github.com/llvm/llvm-project/issues/51311. The core issue is that Rust promises that
-// the drop_in_place() function can't unwind, but implements it in a way that *can*, because we
-// currently go out of our way to allow longjmps, which also use the unwinding mechanism on MSVC
-// targets. We should either forbid longjmps, or not assume nounwind, making this optimization
-// incompatible with the current behavior of running cleanuppads on longjmp unwinding.
+// https://github.com/llvm/llvm-project/issues/51311.
 
 // CHECK-NOT: {{(call|invoke).*}}should_not_appear_in_output