[PATCH] D144999: [Clang][MC][MachO]Only emits compact-unwind format for "canonical" personality symbols. For the rest, use DWARFs.

2023-06-08 Thread Vy Nguyen via Phabricator via cfe-commits
oontvoo added a comment. In D144999#4405793 , @fdeazeve wrote: > I believe this is breaking the LLDB tests: > https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/56106/changes > https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/buildTimeTre

[PATCH] D144999: [Clang][MC][MachO]Only emits compact-unwind format for "canonical" personality symbols. For the rest, use DWARFs.

2023-06-08 Thread Vy Nguyen via Phabricator via cfe-commits
oontvoo added a comment. P.S: https://reviews.llvm.org/D152449 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144999/new/ https://reviews.llvm.org/D144999 ___ cfe-commits mailing list cfe-commits@lists.ll

[PATCH] D144999: [Clang][MC][MachO]Only emits compact-unwind format for "canonical" personality symbols. For the rest, use DWARFs.

2023-06-09 Thread Vy Nguyen via Phabricator via cfe-commits
oontvoo added a comment. In D144999#4407934 , @t.p.northover wrote: > I don't think this handles the no-personality case properly. For example this > code leads to a DWARF entry now: > > void bar(int *) noexcept; > void foo() { > int arr; >

[PATCH] D144999: [Clang][MC][MachO]Only emits compact-unwind format for "canonical" personality symbols. For the rest, use DWARFs.

2023-06-09 Thread Vy Nguyen via Phabricator via cfe-commits
oontvoo added a comment. In D144999#4409149 , @Michael137 wrote: > In D144999#4408248 , @Michael137 > wrote: > >> Looks like the latest reland of this patch >> (`e60b30d5e3878e7d91f8872ec4c4dca00d4a2dfc`) broke

[PATCH] D144999: [Clang][MC][MachO]Only emits compact-unwind format for "canonical" personality symbols. For the rest, use DWARFs.

2023-06-09 Thread Vy Nguyen via Phabricator via cfe-commits
oontvoo added a comment. P.S This might have the same root-cause with the previous comment and could be fixed by D152540 as well. (not able to repro the failures yet - don't have arm64 readily available, will test it later this evening. but on linux x86-64 and

[PATCH] D144999: [Clang][MC][MachO]Only emits compact-unwind format for "canonical" personality symbols. For the rest, use DWARFs.

2023-06-12 Thread Vy Nguyen via Phabricator via cfe-commits
oontvoo added a comment. In D144999#4415137 , @jasonmolenda wrote: > Michael and I looked into this. This simple c++ file is resulting in > eh_frame unwind info on aarch64 darwin instead of compact unwind info. The > eh_frame instructions don't descr

[PATCH] D144999: [Clang][MC][MachO]Only emits compact-unwind format for "canonical" personality symbols. For the rest, use DWARFs.

2023-06-13 Thread Vy Nguyen via Phabricator via cfe-commits
oontvoo added a comment. In D144999#4415419 , @oontvoo wrote: > In D144999#4415137 , @jasonmolenda > wrote: > >> Michael and I looked into this. This simple c++ file is resulting in >> eh_frame unwind info on a

[PATCH] D139652: Add the thread sanitizer support for X86_64 WatchOS simulators

2022-12-09 Thread Vy Nguyen via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG0073fd8d0dda: Add the thread sanitizer support for X86_64 WatchOS simulators (authored by yfyang, committed by oontvoo). Repository: rG LLVM Githu

<    1   2