[libunwind] Ensure test/libunwind_01.pass is not completely inlined
authorAlex Richardson <Alexander.Richardson@cl.cam.ac.uk>
Mon, 20 Jun 2022 08:57:12 +0000 (08:57 +0000)
committerAlex Richardson <alexrichardson@google.com>
Mon, 20 Jun 2022 09:05:49 +0000 (09:05 +0000)
commit8df257a6d0b0b32e05e89874eff4c8ddd2a2a2a9
tree8864ace3b8a3835cfed86de3ce2991ec224b4246
parentb02d970b4335561940dd584f6e6497ab684c788d
[libunwind] Ensure test/libunwind_01.pass is not completely inlined

By adding noinline and calling fprintf before returning we ensure that
every function will have a distinct call frame and that the return address
will always be saved instead of saving the target in main as the result.

Before this change all backtraces were always backtrace -> main -> _start,
i.e. always exactly three entries. This happenend because all calls were
inlined in main() and the test just happenend to pass because there is at
least _start before main.

I found this while fixing some bugs in libunwind for CHERI and noticed that
the test was passing even though the code was completely broken.

Obtained from: https://github.com/CTSRD-CHERI/llvm-project

Reviewed By: #libunwind, ldionne, MaskRay

Differential Revision: https://reviews.llvm.org/D126611
libunwind/test/libunwind_01.pass.cpp