[PATCH] D39280: [libunwind] Use uint64_t for unw_word_t in ARM EHABI

2017-10-30 Thread Martin Storsjö via Phabricator via cfe-commits
mstorsjo abandoned this revision. mstorsjo added a comment. This was made obsolete by https://reviews.llvm.org/D39365. https://reviews.llvm.org/D39280 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D39280: [libunwind] Use uint64_t for unw_word_t in ARM EHABI

2017-10-27 Thread Martin Storsjö via Phabricator via cfe-commits
mstorsjo added a comment. In https://reviews.llvm.org/D39280#908825, @compnerd wrote: > I dont think that this is correct. IIRC, the unwind specification expects > `unw_word_t` to match `uintptr_t` (that is, it should be 32-bits on > https://reviews.llvm.org/P32 environments). I checked

[PATCH] D39280: [libunwind] Use uint64_t for unw_word_t in ARM EHABI

2017-10-26 Thread Martin Storsjö via Phabricator via cfe-commits
mstorsjo added a comment. In https://reviews.llvm.org/D39280#908825, @compnerd wrote: > I dont think that this is correct. IIRC, the unwind specification expects > `unw_word_t` to match `uintptr_t` (that is, it should be 32-bits on > https://reviews.llvm.org/P32 environments). Ok, well

[PATCH] D39280: [libunwind] Use uint64_t for unw_word_t in ARM EHABI

2017-10-26 Thread Saleem Abdulrasool via Phabricator via cfe-commits
compnerd added a comment. I dont think that this is correct. IIRC, the unwind specification expects `unw_word_t` to match `uintptr_t` (that is, it should be 32-bits on https://reviews.llvm.org/P32 environments). https://reviews.llvm.org/D39280

[PATCH] D39280: [libunwind] Use uint64_t for unw_word_t in ARM EHABI

2017-10-25 Thread Martin Storsjö via Phabricator via cfe-commits
mstorsjo created this revision. Herald added subscribers: kristof.beyls, aprantl, aemerson. This reduces the differences between the EHABI and DWARF cases. This allows getting rid of some casts in _LIBUNWIND_TRACE_API log lines, making them match what's in UnwindLevel1.c for DWARF. The only