[PATCH] Enable UBSan on Windows

Saleem Abdulrasool compnerd at compnerd.org
Fri Oct 24 18:50:28 PDT 2014


================
Comment at: cmake/config-ix.cmake:205
@@ -205,1 +204,3 @@
+    OS_NAME MATCHES "Darwin|Linux|FreeBSD" OR
+    (OS_NAME MATCHES "Windows" AND CMAKE_SIZEOF_VOID_P EQUAL 4))
   set(COMPILER_RT_HAS_UBSAN TRUE)
----------------
akhaldi wrote:
> timurrrr wrote:
> > have you tried UBSan on Win64?
> No, which is why I kept the 32bit check for now.
What about WoA (32-bit)?

http://reviews.llvm.org/D5368






More information about the llvm-commits mailing list