[PATCH] [libcxxabi] Fix -Werror build for 32-bit non-ARM.

Renato Golin renato.golin at linaro.org
Sat Feb 14 11:36:10 PST 2015


The idea of making this header public may be worrying, if we end up requiring the user to include it. If this is just a matter of compiler-logic, than it should be fine.


================
Comment at: src/Unwind/Unwind-EHABI.h:16
@@ +15,3 @@
+
+#if LIBCXXABI_ARM_EHABI
+
----------------
danalbert wrote:
> rengolin wrote:
> > Is this duplication intended?
> Bad merge after splitting/rebasing my change. It was fixed right after in a follow up patch.
I can't see it. Has this change been merged already?

http://reviews.llvm.org/D7419

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list