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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
