[PATCH] D21991: [libunwind][ARM] Improve unwinder stack usage - Make WMMX support optional

[Asiri Rathnayake via cfe-commits]

rmaprath added inline comments.

================
Comment at: src/Registers.hpp:1497
@@ -1479,3 +1496,3 @@
     _LIBUNWIND_ABORT("unsupported arm register");
 }
 
----------------
compnerd wrote:
> Early returns would be nicer imo.
Not sure if I follow, did you mean to check the bounds of `regNum` the first thing and `_LIBUNWIND_ABORT` sooner than later? Might convolute the code given the conditional on `__ARM_WMMX` and I'm not sure what benefit it brings? Or perhaps I misunderstood you?

I'll commit the current patch as it is for the moment and then do a clean-up (once I understand what you mean).

Thanks.


http://reviews.llvm.org/D21991