[PATCH] D46971: [DWARF] Get RA from RA register even if it appears unused

whitequark via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 29 19:52:31 PDT 2018


whitequark added a comment.

> It would be nice if we had a test case added for this

This is inherently architecture and platform specific, and libunwind currently doesn't have any provisions for this... In principle, would a x86-only Linux-only test suffice? I'm not sure if I can spend time making it more portable.



================
Comment at: src/DwarfInstructions.hpp:195
         }
+        else if (i == (int)cieInfo.returnAddressRegister)
+          returnAddress = registers.getRegister(i);
----------------
compnerd wrote:
> I think that we should stick to LLVM style (coddled braces) and use C++ style casts.
Ack re braces.

I was copying that cast from ten lines above. Should I change that too? This should be a static_cast, right?


Repository:
  rUNW libunwind

https://reviews.llvm.org/D46971





More information about the cfe-commits mailing list