public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [Patch ObjC/Committed] fix PR47997 (part 2).
@ 2011-10-29 15:09 Iain Sandoe
  0 siblings, 0 replies; only message in thread
From: Iain Sandoe @ 2011-10-29 15:09 UTC (permalink / raw)
  To: GCC Patches; +Cc: Mike Stump, Nicola Pero

[-- Attachment #1: Type: text/plain, Size: 36 bytes --]

As approved on the PR thread,
Iain


[-- Attachment #2: 180650-pr467997-objc-diff.txt --]
[-- Type: text/plain, Size: 1123 bytes --]

Index: gcc/objc/ChangeLog
===================================================================
--- gcc/objc/ChangeLog	(revision 180650)
+++ gcc/objc/ChangeLog	(working copy)
@@ -1,3 +1,10 @@
+2011-10-29  Iain Sandoe  <iains@gcc.gnu.org>
+
+	PR target/47997
+	* objc-act.c (objc_build_string_object): Remove redundant second
+	call to fix_string_type ().  Add a checking assert that we are,
+	indeed, passed a STRING_CST.
+
 2011-10-18  Mikael Pettersson  <mikpe@it.uu.se>
 
 	PR objc/50743
Index: gcc/objc/objc-act.c
===================================================================
--- gcc/objc/objc-act.c	(revision 180650)
+++ gcc/objc/objc-act.c	(working copy)
@@ -3128,9 +3128,8 @@ objc_build_string_object (tree string)
   struct string_descriptor *desc, key;
   void **loc;
 
-  /* Prep the string argument.  */
-  string = fix_string_type (string);
-  TREE_SET_CODE (string, STRING_CST);
+  /* We should be passed a STRING_CST.  */
+  gcc_checking_assert (TREE_CODE (string) == STRING_CST);
   length = TREE_STRING_LENGTH (string) - 1;
 
   /* The target may have different ideas on how to construct an ObjC string

[-- Attachment #3: Type: text/plain, Size: 1 bytes --]



^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2011-10-29 13:01 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-10-29 15:09 [Patch ObjC/Committed] fix PR47997 (part 2) Iain Sandoe

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).