builtins.c (build_string_literal): Avoid generating a non-gimple_val result.

2008-05-30  Richard Guenther  <rguenther@suse.de>

	* builtins.c (build_string_literal): Avoid generating
	a non-gimple_val result.

From-SVN: r136222

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 9d29edca4fc92c81ba1d53808788565a28a9cbfd..82739e03eff3666f1ae3821ef76fb1d57976524f 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2008-05-30  Richard Guenther  <rguenther@suse.de>
+
+       * builtins.c (build_string_literal): Avoid generating
+       a non-gimple_val result.
+
 2008-05-30  DJ Delorie  <dj@redhat.com>
 
        * exec-tool.in: Use an environment variable (private) instead of a

diff --git a/gcc/builtins.c b/gcc/builtins.c
index 695567cfbfc8e0981698991f0d305b11232f7e8f..77cd48beaed6924dcf602e0500e517ec5ea04d01 100644 (file)
--- a/gcc/builtins.c
+++ b/gcc/builtins.c
@@ -5249,18 +5249,17 @@ build_string_literal (int len, const char *str)
 
   t = build_string (len, str);
   elem = build_type_variant (char_type_node, 1, 0);
-  index = build_index_type (build_int_cst (NULL_TREE, len - 1));
+  index = build_index_type (size_int (len - 1));
   type = build_array_type (elem, index);
   TREE_TYPE (t) = type;
   TREE_CONSTANT (t) = 1;
   TREE_READONLY (t) = 1;
   TREE_STATIC (t) = 1;
 
-  type = build_pointer_type (type);
-  t = build1 (ADDR_EXPR, type, t);
-
   type = build_pointer_type (elem);
-  t = build1 (NOP_EXPR, type, t);
+  t = build1 (ADDR_EXPR, type,
+             build4 (ARRAY_REF, elem,
+                     t, integer_zero_node, NULL_TREE, NULL_TREE));
   return t;
 }