[PATCH] Activate UBSan for Power

Bill Seurer seurer at linux.vnet.ibm.com
Tue Mar 31 14:06:22 PDT 2015


================
Comment at: projects/compiler-rt/test/ubsan/TestCases/Float/cast-overflow.cpp:91
@@ -90,1 +90,3 @@
+    return 0;
+    }
   case '1':
----------------
wschmidt wrote:
> Probably should make test_int static or filescope, as otherwise the compiler is within its rights to optimize this away despite the exception.  Same comment applies several more times below.
Originally I had them as volatile but that changed the types in the error messages so I dropped it.  I will switch them to static.

http://reviews.llvm.org/D8743

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






More information about the llvm-commits mailing list