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

Asiri Rathnayake via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 5 09:13:47 PDT 2016


rmaprath added inline comments.

================
Comment at: CMakeLists.txt:108
@@ -107,2 +107,3 @@
 option(LIBUNWIND_ENABLE_CROSS_UNWINDING "Enable cross-platform unwinding support." OFF)
+option(LIBUNWIND_ENABLE_ARM_WMMX "Enable unwinding support for ARM WMMX registers." ON)
 
----------------
bcraig wrote:
> Buried in one of the .s files, I see a preprocessor definition check for __ARM_WMMX.  Can the use of the preprocessor define be used instead of adding a new configuration option?
Missed that. Digging into some past commits bumped into http://reviews.llvm.org/D5314#74501

I think it's safe to guard the whole thing with `__ARM_WMMX`. Will do that instead.




http://reviews.llvm.org/D21991





More information about the cfe-commits mailing list