[PATCH] Activate UBSan for Power

Bill Schmidt wschmidt at linux.vnet.ibm.com
Tue Mar 31 13:55:47 PDT 2015


One small concern inline...


================
Comment at: projects/compiler-rt/test/ubsan/TestCases/Float/cast-overflow.cpp:91
@@ -90,1 +90,3 @@
+    return 0;
+    }
   case '1':
----------------
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.

http://reviews.llvm.org/D8743

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






More information about the llvm-commits mailing list