[PATCH] [libcxxabi] Fix -Werror build for 32-bit non-ARM.
Saleem Abdulrasool
compnerd at compnerd.org
Thu Feb 5 14:52:42 PST 2015
LGTM with the first suggestion (about readability). I am less worried if you want to merge the drive-by formatting cleanup rather than separating it.
================
Comment at: include/__cxxabi_config.h:13
@@ +12,3 @@
+
+#if !defined(__USING_SJLJ_EXCEPTIONS__) && defined(__arm__) && \
+ !defined(__ARM_DWARF_EH__) && !defined(__APPLE__)
----------------
I think this is more readable as:
#if defined(__arm__) && !defined(__USING_SJLJ_EXCEPTIONS__) && !defined(__ARM_DWARF_EH__)
That maps to what the condition really is: you need ARM, non-SJLJ, non-DWARF exceptions. Apple uses SJLJ exceptions, so adding `!defined(__APPLE__)` seems ... repetitive.
================
Comment at: src/Unwind/UnwindCursor.hpp:26
@@ -25,3 +25,3 @@
-#include "libunwind.h"
+#include "config.h"
----------------
Unless Im mistaken, this part looks like it could be split out into a separate change.
================
Comment at: src/cxa_exception.cpp:263
@@ -262,5 +262,3 @@
#else
- return cxa_exception_from_exception_unwind_exception
- (
- static_cast<_Unwind_Exception*>(unwind_exception)
- )->adjustedPtr;
+ return cxa_exception_from_exception_unwind_exception(
+ static_cast<_Unwind_Exception*>(unwind_exception))->adjustedPtr;
----------------
Along with the previous.
================
Comment at: src/cxa_exception.hpp:27
@@ -26,4 +26,3 @@
static const uint64_t kOurDependentExceptionClass = 0x434C4E47432B2B01; // CLNGC++\1
-static const uint64_t get_vendor_and_language = 0xFFFFFFFFFFFFFF00; // mask for CLNGC++
-
-struct __cxa_exception {
+static const uint64_t get_vendor_and_language = 0xFFFFFFFFFFFFFF00; // mask for CLNGC++
+
----------------
And this.
http://reviews.llvm.org/D7419
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list