[PATCH] D39074: [libunwind][MIPS]: Add support for unwinding in N32 processes.

Saleem Abdulrasool via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 4 17:38:06 PST 2018


compnerd added a comment.

Looking over this patch again, I think I really would prefer that this was split up into two patches.  The first one should be entirely mechanical, replacing `n64` with `newabi`.  The second patch would actually make the changes that you are are after.  That would really help with focusing what the issue here actually is.  I don't see anything technically that is an issue (I admit I didn't verify the sizes, but the assertions should hopefully catch that).  Beyond that split up, Id like to get a signoff from @sdardis for the MIPS specific bits, but from the libunwind side, LGTM.


https://reviews.llvm.org/D39074





More information about the cfe-commits mailing list