[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-07-26 Thread David Tenty via cfe-commits
daltenty wrote: This is breaking a number of tests for the UBSAN on AIX, the traceback produced by the runtime are no longer correct. Before this change: ``` UndefinedBehaviorSanitizer:DEADLYSIGNAL ==52887862==ERROR: UndefinedBehaviorSanitizer: SEGV on unknown address 0x (pc 0x1784

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-07-16 Thread Louis Dionne via cfe-commits
ldionne wrote: Is it possible to add a test case for this? https://github.com/llvm/llvm-project/pull/92291 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-07-09 Thread Saleem Abdulrasool via cfe-commits
https://github.com/compnerd closed https://github.com/llvm/llvm-project/pull/92291 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-07-09 Thread Azat Khuzhin via cfe-commits
@@ -365,7 +365,12 @@ int DwarfInstructions::stepWithDwarf(A &addressSpace, pint_t pc, // Return address is address after call site instruction, so setting IP to // that does simulates a return. - newRegisters.setIP(returnAddress); + // + // In case

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-07-09 Thread Azat Khuzhin via cfe-commits
https://github.com/azat updated https://github.com/llvm/llvm-project/pull/92291 >From 7066c0d3c7e84ccfdaeebd3b279bcdd91955ec7b Mon Sep 17 00:00:00 2001 From: Azat Khuzhin Date: Wed, 15 May 2024 08:17:22 +0200 Subject: [PATCH] [libunwind] fix unwinding from signal handler In case of this is fram

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-07-08 Thread Azat Khuzhin via cfe-commits
@@ -365,7 +365,12 @@ int DwarfInstructions::stepWithDwarf(A &addressSpace, pint_t pc, // Return address is address after call site instruction, so setting IP to // that does simulates a return. - newRegisters.setIP(returnAddress); + // + // In case

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-07-08 Thread Saleem Abdulrasool via cfe-commits
@@ -365,7 +365,12 @@ int DwarfInstructions::stepWithDwarf(A &addressSpace, pint_t pc, // Return address is address after call site instruction, so setting IP to // that does simulates a return. - newRegisters.setIP(returnAddress); + // + // In case

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-07-07 Thread Azat Khuzhin via cfe-commits
@@ -365,7 +365,12 @@ int DwarfInstructions::stepWithDwarf(A &addressSpace, pint_t pc, // Return address is address after call site instruction, so setting IP to // that does simulates a return. - newRegisters.setIP(returnAddress); + // + // In case

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-07-07 Thread Saleem Abdulrasool via cfe-commits
@@ -365,7 +365,12 @@ int DwarfInstructions::stepWithDwarf(A &addressSpace, pint_t pc, // Return address is address after call site instruction, so setting IP to // that does simulates a return. - newRegisters.setIP(returnAddress); + // + // In case

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-07-07 Thread Azat Khuzhin via cfe-commits
@@ -365,7 +365,12 @@ int DwarfInstructions::stepWithDwarf(A &addressSpace, pint_t pc, // Return address is address after call site instruction, so setting IP to // that does simulates a return. - newRegisters.setIP(returnAddress); + // + // In case

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-07-06 Thread Saleem Abdulrasool via cfe-commits
@@ -365,7 +365,12 @@ int DwarfInstructions::stepWithDwarf(A &addressSpace, pint_t pc, // Return address is address after call site instruction, so setting IP to // that does simulates a return. - newRegisters.setIP(returnAddress); + // + // In case

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-07-05 Thread Azat Khuzhin via cfe-commits
@@ -365,7 +365,12 @@ int DwarfInstructions::stepWithDwarf(A &addressSpace, pint_t pc, // Return address is address after call site instruction, so setting IP to // that does simulates a return. - newRegisters.setIP(returnAddress); + // + // In case

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-07-05 Thread Saleem Abdulrasool via cfe-commits
@@ -365,7 +365,12 @@ int DwarfInstructions::stepWithDwarf(A &addressSpace, pint_t pc, // Return address is address after call site instruction, so setting IP to // that does simulates a return. - newRegisters.setIP(returnAddress); + // + // In case

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-07-05 Thread Azat Khuzhin via cfe-commits
azat wrote: Can someone take a look? Maybe @MaskRay @compnerd (according to the most frequent reviewers from `Reviewed By` tag) https://github.com/llvm/llvm-project/pull/92291 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-05-16 Thread Azat Khuzhin via cfe-commits
azat wrote: >[Build and Test libc++ / stage3 (generic-ubsan, libcxx-runners-8-set) >(pull_request) Failing after >10m](https://github.com/llvm/llvm-project/actions/runs/9099207645/job/25013031369?pr=92291) Looks like unrelated? ``` 2024-05-15T17:48:22.3468783Z ##[error]The runner has received

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-05-15 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-libunwind Author: Azat Khuzhin (azat) Changes In case of this is frame of signal handler, the IP should be incremented, because the IP saved in the signal handler points to first non-executed instruction, while FDE/CIE expects IP to be after the first

[libunwind] [libunwind] fix unwinding from signal handler (PR #92291)

2024-05-15 Thread Azat Khuzhin via cfe-commits
https://github.com/azat created https://github.com/llvm/llvm-project/pull/92291 In case of this is frame of signal handler, the IP should be incremented, because the IP saved in the signal handler points to first non-executed instruction, while FDE/CIE expects IP to be after the first non-execu