From: David Malcolm <dmalcolm@redhat.com>
Date: Tue, 23 Jan 2018 11:05:21 +0000 (+0000)
Subject: Add more test coverage to selftest::test_location_wrappers
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=02851c8c7f8b04a4a726e3162177a5ea4d19ca36;p=gcc.git

Add more test coverage to selftest::test_location_wrappers

gcc/ChangeLog:
	* tree.c (selftest::test_location_wrappers): Add more test
	coverage.

From-SVN: r256979
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 3a4a97be236..a845feff6aa 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2018-01-23  David Malcolm  <dmalcolm@redhat.com>
+
+	* tree.c (selftest::test_location_wrappers): Add more test
+	coverage.
+
 2018-01-23  David Malcolm  <dmalcolm@redhat.com>
 
 	* sbitmap.c (selftest::test_set_range): Fix memory leaks.
diff --git a/gcc/tree.c b/gcc/tree.c
index 452d3855a77..63a29f4868c 100644
--- a/gcc/tree.c
+++ b/gcc/tree.c
@@ -14492,6 +14492,8 @@ test_location_wrappers ()
 {
   location_t loc = BUILTINS_LOCATION;
 
+  ASSERT_EQ (NULL_TREE, maybe_wrap_with_location (NULL_TREE, loc));
+
   /* Wrapping a constant.  */
   tree int_cst = build_int_cst (integer_type_node, 42);
   ASSERT_FALSE (CAN_HAVE_LOCATION_P (int_cst));
@@ -14502,6 +14504,14 @@ test_location_wrappers ()
   ASSERT_EQ (loc, EXPR_LOCATION (wrapped_int_cst));
   ASSERT_EQ (int_cst, tree_strip_any_location_wrapper (wrapped_int_cst));
 
+  /* We shouldn't add wrapper nodes for UNKNOWN_LOCATION.  */
+  ASSERT_EQ (int_cst, maybe_wrap_with_location (int_cst, UNKNOWN_LOCATION));
+
+  /* We shouldn't add wrapper nodes for nodes that CAN_HAVE_LOCATION_P.  */
+  tree cast = build1 (NOP_EXPR, char_type_node, int_cst);
+  ASSERT_TRUE (CAN_HAVE_LOCATION_P (cast));
+  ASSERT_EQ (cast, maybe_wrap_with_location (cast, loc));
+
   /* Wrapping a STRING_CST.  */
   tree string_cst = build_string (4, "foo");
   ASSERT_FALSE (CAN_HAVE_LOCATION_P (string_cst));