[libcxx-commits] [PATCH] D77359: [libunwind] add hexagon support

[Martin Storsjö via Phabricator via libcxx-commits]

mstorsjo added inline comments.


================
Comment at: libunwind/src/UnwindRegistersRestore.S:853
+  jumpr r31
+.size _ZN9libunwind17Registers_hexagon6jumptoEv, .-_ZN9libunwind17Registers_hexagon6jumptoEv
 #elif defined(__mips__) && defined(_ABIO32) && _MIPS_SIM == _ABIO32
----------------
There's no other `.size` directives used for other ELF platforms as far as I can see, and you aren't adding one in UnwindRegistersSave.S - so this feels a bit inconsistent.


================
Comment at: libunwind/src/UnwindRegistersSave.S:1109
   WEAK_ALIAS(__unw_getcontext, unw_getcontext)
+#endif
 
----------------
What's the purpose and need of handling the weak alias differently here? Does the assembler use a different syntax for the weak things there, or do you apply weak to the symbol `__unw_getcontext` as well?

I think it might be better to move this difference into `assembly.h` to encapsulate the implementation details of how to set up a weak alias there (there's three different cases there already).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77359