[committed] Mark cr16 as keeping null pointer checks in target-supports.exp

Jeff Law law@redhat.com
Tue Apr 23 15:04:00 GMT 2019


My tester showed a spurious regression with a recent patch on cr16.  In
reviewing the logs I saw unexpected results for the proposed changes to
isolate-2.c and isolate-4.c.

The cr16 turns off NULL pointer check elimination, but isn't listed as
doing so in target-supports.exp.  As a result we run the tests, but they
fail.

This patch adds cr16 to the list of targets that keep null pointer
checks and those tests are now skipped.

I'm installing this on the trunk momentarily.

Jeff
-------------- next part --------------
	* lib/target-supports.exp
	(check_effective_target_keeps_null_pointer_checks): Add cr16.

diff --git a/gcc/testsuite/lib/target-supports.exp b/gcc/testsuite/lib/target-supports.exp
index 0dc1e184f55..4bb7da283f1 100644
--- a/gcc/testsuite/lib/target-supports.exp
+++ b/gcc/testsuite/lib/target-supports.exp
@@ -559,7 +559,7 @@ proc check_effective_target_keeps_null_pointer_checks { } {
     if [target_info exists keeps_null_pointer_checks] {
       return 1
     }
-    if { [istarget msp430-*-*] } {
+    if { [istarget msp430-*-*] || [istarget cr16-*-*] } {
 	return 1;   
     }
     return 0


More information about the Gcc-patches mailing list