[libcxx-commits] [PATCH] D70397: [libunwind] Adjust the signal_frame test for Arm

[Sergej Jaskiewicz via Phabricator via libcxx-commits]

broadwaylamb added a comment.

In D70397#1753767 <https://reviews.llvm.org/D70397#1753767>, @steven_wu wrote:

> Can you post the compile commands used by the test? `.cfi_startproc` and `.cfi_endproc` should be there for most platforms if the test is built with -funwind-table.


I'm cross-compiling on Windows for ARM Linux (arm-linux-gnueabihf), so I'm using ARM EHABI.

Here's the complete command (I've reduced the number of flags from the actual test invocation, but believe me, those were unrelated, and the error was still produced there):

  bin\clang.exe -funwind-tables -I..\libunwind\include -S ..\libunwind\test\signal_frame.pass.cpp

It produces the error. And indeed, if I remove the `asm(".cfi_signal_frame");` line from the test (to get it to compile), the generated assembly doesn't contain `.cfi_startproc` and `.cfi_endproc`.

**However**, manually adding the `-g` flag helps, and those directives are generated. But the `-g` flag is not enabled for libunwind tests and I wonder if it should be. If it shouldn't, how can we force those directives to be generated?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70397