jroelofs added inline comments.

================
Comment at: src/AddressSpace.hpp:521
                                                 unw_word_t *offset) {
-#ifndef _LIBUNWIND_IS_BAREMETAL
+#if !defined(_LIBUNWIND_IS_BAREMETAL) && !defined(_WIN32)
   Dl_info dyldInfo;
----------------
Would it work to implement the win32 side of this via `SymFromAddr`?


================
Comment at: src/UnwindRegistersRestore.S:29
 #  +                       +
+#if !defined(_WIN32)
   movl   4(%esp), %eax
----------------
Please invert the condition, and swap the if/else on this. That will make it 
more straightforward to adjust this for other platforms later.


https://reviews.llvm.org/D38679



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to