[PATCH] D121017: [AArch64] Fallback to DWARF when trying to emit compact unwind info with multiple CFA offset adjustments

Tim Northover via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 26 13:34:35 PDT 2022


t.p.northover added a comment.

I don't think this is the ideal fix. We don't do asynchronous unwind on Darwin, and for synchronous cases it looked to me like the final stack update was still correct. So rather than pessimizing lots and lots of functions' unwind tables (a significant size impact), I think it's better to simply disable the assertion: https://reviews.llvm.org/D121116 (I'll update it for the status quo with tests on Monday).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D121017/new/

https://reviews.llvm.org/D121017



More information about the llvm-commits mailing list